Integrate Element Templates Runtime Versions

[jarekdanielak]

Proposed Changes

Integrate element templates runtime version compatibility.

Camunda 8 BPMN editor will now pass current runtime version to element templates module. If template is incompatible, warning will be shown in properties panel and problems panel.

To try it out, use the new "Compatibility test" template from cloud-samples and try changing template version in XML or Camunda version.

Closes #4617
Closes #4648
Closes #4530
Closes #4749

Checklist

To ensure you provided everything we need to look at your PR:

• Brief textual description of the changes present
• Visual demo attached
• Steps to try out present, i.e. using the @bpmn-io/sr tool
• Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

[jarekdanielak]

Styles for Incompatible button are missing, but this will be fixed with dependencies update in camunda-bpmn-js: camunda/camunda-bpmn-js#396

[nikku]

Can we / should we test this?

[jarekdanielak]

Styles problem is fixed with update to [email protected].

Can we / should we test this?

The test suite for modeling features is quite slim. Do we want to work on extending it as part of this PR?

[jarekdanielak]

cc. @lmbateman for final look at the UI.

[nikku]

The test suite for modeling features is quite slim. Do we want to work on extending it as part of this PR?

Let's ensure this does not break accidentally in the future. I don't care which path we take to accomplish this 🙂

[nikku]

Will resolve conflicts now, and test manually.

[nikku]

Rebased on develop, marked related issues (fixed through the version bumps).

[nikku]

Works on my machine.

@jarekdanielak consider to follow-up (in any way) with this.

[lmbateman]

Checked using npx @bpmn-io/sr bpmn-io/bpmn-js-element-templates#main

✅ Both buttons look good
✅ Both dropdowns have the right text and options
✅ Lint warnings look good

In general, it looks great to me. Two very minor items to think about:

• "Update" and "Unlink" don't look like actions to me (for all issues, not just Incompatible). Yana's original designs had them in blue. Since this affects things outside this story, I'm not sure it's in scope for now.
• Would it make sense to put a v in front of the version? When it's a long string, it's probably fine, but the 1 in the demo confuses me slightly. Version? Error code? Typo? :)

[jarekdanielak]

@lmbateman the feature is already merged, but I will save your comment as a follow up for later. Thanks for the feedback!