-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add subprocess with start event included #1039
Conversation
9e0fb1d
to
af2c22d
Compare
81607d2
to
d35dbce
Compare
Looks good. |
d35dbce
to
aff8315
Compare
var shape = event.context.shape, | ||
hints = event.context.hints; | ||
|
||
if ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be something like hints.shouldAddStartEvent = is('subporcess') && isExpanded(shape)?
That piece of code will set shouldaddStartEvent for every task.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, that simplifies things.
When: 1. Creating expanded subprocess from palette. 2. Replacing task with expanded subprocess.
aff8315
to
ff31bfe
Compare
Which issue does this PR address?
Related to camunda/camunda-modeler#1242
Dependent on bpmn-io/diagram-js#347
This automatically adds a start event to an expanded subprocess when: