Model converter - conditional events variableNamesFilter is not supported

[venetrius]

Acceptance Criteria (Required on creation)

• Variable names filter is not present in the transformed C8 BPMN
• A message is present to highlight the differences in Conditional Event evaluation in Camunda 7 and Camunda 8.

Hints

Variable names filter is not supported by C8.
Changes: variableNames filter is removed, the FEEL expression defines the dependency, for example the expression x > 10 only re-evaluates when x changes.

Links

• Context for the scope change

Breakdown

Pull Requests

• feat(diagram-converter): variableName filter not supported #1045

Dev2QA handover

• Does this ticket need a QA test, and are the testing goals not clear from the description? Add a Dev2QA handover comment

[venetrius]

Decisions

When should a message be added to the converted bpmn?

Options

Always

• Based on C7 BPMN contains variableName filter or not show different message:
  • "Variable name filter is not supported in conditional event on '%s'. Camunda 8 calculates when to evaluate the FEEL expression based on the expression itself.",
  • "The condition expression on conditional event '%s' will be evaluated on variable events present in the expression."

Only when C7 BPMN contains variableName filter

• "Variable name filter is not supported in conditional event on '%s'. Camunda 8 calculates when to evaluate the FEEL expression based on the expression itself.",

Selected solution. - Only when C7 BPMN contains variableName filter

Reasoning: Showing the message all the time would increase noise, without providing a lot of value. If no variable name filter was used in Camunda 7 the user experience does not change when using Camunda 8.
Implementation wise it is not a big difference in effort.