search

camunda / camunda-modeler

An integrated modeling solution for BPMN, DMN and Forms based on bpmn.io.
https://camunda.com/products/modeler
MIT License
1.51k stars 484 forks source link

Cleanup CMMN from the modeler, or at least remove the option to enable it through flags. #3033

Open Skaiir opened 2 years ago

Skaiir commented 2 years ago

What should we do?

We should decide on what we want to do with the leftover CMMN code in the codebase. It is still possible to enable the editor via the --no-disable-cmmn flag, however the editor is no longer functional in our UI, and we do not maintain it anymore.

Hence I would say we should at the very least remove the flag, or only expose it to developers (if that's a possibility) if through some miracle we want to resurrect CMMN

Why should we do it?

The CMMN editor is no longer maintained, and currently crashing. It also conflicts with the UI and hence should be at least hidden away.

nikku commented 2 years ago

We should decide on what we want to do with the leftover CMMN code in the codebase. It is still possible to enable the editor via the --no-disable-cmmn flag, however the editor is no longer functional in our UI, and we do not maintain it anymore.

The actual editor is broken you say? In this case we could consider removing the flag alltogether.

Skaiir commented 2 years ago

@nikku Yes I believe due to breaking changes to the properties panel relating to groups.