search

bpmn-io / bpmn-js-element-templates

The element template extension for bpmn-js
MIT License
5 stars 3 forks source link

Verify #78 #80

Closed nikku closed 7 months ago

nikku commented 7 months ago

Closes #78

nikku commented 7 months ago

Observation: We added a circuit breaker in the update behavior via https://github.com/bpmn-io/bpmn-js-element-templates/commit/f1aa3a3d1d8712d4d1a7c0bc22a89c6e953e4725 just to remove it (again) via https://github.com/bpmn-io/bpmn-js-element-templates/commit/3e237c12725f0bddc950b5d77403fabfe1830bcb.

nikku commented 7 months ago

:brain: I've spent some time yesterday evening to investigate the situation. What is on my mind right now:

philippfromme commented 7 months ago

👍🏻 for testing all connector templates by default, though I'd expect the connectors to have automated tests for this.

nikku commented 7 months ago

Updated with other test including a minimal repro for #78: https://github.com/bpmn-io/bpmn-js-element-templates/pull/80/commits/3683975939c26f9e02d656212a081cc6f4ea7722.