Skip to content
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

fix(create-participant-behavior): ensure available children #1143

Merged
merged 1 commit into from
Aug 5, 2019

Conversation

pinussilvestrus
Copy link
Contributor

This ensures we have left children to properly create new participant's bounds. This case of no left children after the filter operation can happen if we only have bpmn:Group elements in the diagram (which wouldn't make sense in practice, but however ...)

Closes #1133

Aug-05-2019 09-57-37

* Ensure there are leftover children to calculate createConstraints

Closes #1133
@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label Aug 5, 2019
@barmac
Copy link
Member

barmac commented Aug 5, 2019

As a bugfix, this can (and should) target the master branch. Can you fix this?

@pinussilvestrus pinussilvestrus changed the base branch from develop to master August 5, 2019 09:51
@pinussilvestrus
Copy link
Contributor Author

Branched it out of master previously, simply chose the wrong branch while creating the pull request 👍

Copy link
Member

@barmac barmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Branched it out of master previously, simply chose the wrong branch while creating the pull request 👍

This happens to everyone :). Anyway, good job!

@merge-me merge-me bot merged commit 5d5f33c into master Aug 5, 2019
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Aug 5, 2019
@delete-merged-branch delete-merged-branch bot deleted the 1133-groups-in-empty-diagrams branch August 5, 2019 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants