search

bpmn-io / extract-process-variables

An util for bpmn-js to extract Camunda BPM process variables from a BPMN 2.0 diagram.
MIT License
0 stars 2 forks source link

Fix zeebe distro #22

Closed marstamm closed 2 years ago

marstamm commented 2 years ago

I found a Problem when integrating the properties panel into the modeler: https://github.com/camunda/camunda-modeler/runs/7633942478?check_suite_focus=true#step:6:309

It seems that package.exports is not supported by webpack and it is using the project folder structure instead of the defined exports (see discussion why we added https://github.com/bpmn-io/extract-process-variables/pull/20#discussion_r923832953)

According to the docs, this should be supported by webpack@5, so we might want to upgrade in the future. For this Release, I created a PR that creates a compatible folder structure during release.

nikku commented 2 years ago

Note that preact handles this nicely in exactly the way you proposed: https://github.com/preactjs/preact

I personally do not see anything bad with doing it like this.