search

bpmn-io / camunda-transaction-boundaries

Visualize all Camunda transaction boundaries in a BPMN diagram.
MIT License
15 stars 9 forks source link

Remove TX boundary from Event-based Gateway #8

Closed falko closed 4 years ago

falko commented 4 years ago

and add TODO about Parallel/Inclusive Gateway with multiple incoming sequence flows

fixes https://github.com/camunda/camunda-modeler-plugins/issues/18

ingorichtsmeier commented 4 years ago

Hi Falko, as we use test driven development here, you should take care about the tests as well.

This line: https://github.com/bpmn-io/camunda-transaction-boundaries/blob/master/test/spec/TransactionBoundariesSpec.js#L122 breaks the build.

falko commented 4 years ago

Thx for the hint. Should be fixed now.

pinussilvestrus commented 4 years ago

Merged via 11e4179522c83de465d35b6a6e401c0ecaced2b9

pinussilvestrus commented 4 years ago

Regarding the TODO, could you maybe open an issue for that so that we are aware of that? @falko