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

Allow Copying Boundary Events #1154

Closed
philippfromme opened this issue Aug 8, 2019 · 4 comments · Fixed by #1192 or #1203
Closed

Allow Copying Boundary Events #1154

philippfromme opened this issue Aug 8, 2019 · 4 comments · Fixed by #1192 or #1203
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers modeling
Milestone

Comments

@philippfromme
Copy link
Contributor

Since detaching of boundary events is now possible it should be possible to copy boundary events. This is only prevented by the element.copy rule.

When disabling this rule:

foobar

@nikku
Copy link
Member

nikku commented Aug 8, 2019

Works? 🐙

@nikku nikku added enhancement New feature or request good first issue Good for newcomers backlog Queued in backlog modeling labels Aug 8, 2019
@nikku
Copy link
Member

nikku commented Aug 8, 2019

Makes a lot of sense, thanks for reporting 👍

@philippfromme
Copy link
Contributor Author

Works? 🐙

Yeah, it works.

@nikku nikku added this to the M31 milestone Sep 9, 2019
@nikku nikku added ready Ready to be worked on and removed backlog Queued in backlog labels Sep 9, 2019
@oguzeroglu oguzeroglu self-assigned this Sep 11, 2019
@oguzeroglu oguzeroglu added in progress Currently worked on and removed ready Ready to be worked on labels Sep 11, 2019 — with bpmn-io-tasks
@oguzeroglu
Copy link
Contributor

oguzeroglu commented Sep 11, 2019

Realised this behaviour while working on this issue.

@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Sep 20, 2019
@fake-join fake-join bot closed this as completed in #1192 Sep 25, 2019
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Sep 25, 2019
@bpmn-io-tasks bpmn-io-tasks bot reopened this Sep 26, 2019
@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Sep 26, 2019
philippfromme pushed a commit to bpmn-io/diagram-js that referenced this issue Sep 26, 2019
philippfromme added a commit that referenced this issue Oct 4, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
oguzeroglu pushed a commit that referenced this issue Oct 7, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
oguzeroglu pushed a commit that referenced this issue Oct 7, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
oguzeroglu pushed a commit that referenced this issue Oct 7, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
oguzeroglu pushed a commit that referenced this issue Oct 7, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
philippfromme added a commit that referenced this issue Oct 10, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
@philippfromme philippfromme added needs review Review pending and removed in progress Currently worked on labels Oct 10, 2019 — with bpmn-io-tasks
philippfromme added a commit that referenced this issue Oct 10, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
philippfromme added a commit that referenced this issue Oct 10, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
barmac pushed a commit that referenced this issue Oct 11, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
philippfromme added a commit that referenced this issue Oct 11, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
philippfromme added a commit that referenced this issue Oct 13, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
Closes #1202
Closes #1204
Closes #1205
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Oct 13, 2019
philippfromme added a commit that referenced this issue Oct 13, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
Closes #1202
Closes #1204
Closes #1205
philippfromme added a commit that referenced this issue Oct 13, 2019
* allow copying boundary events without host
* remove CreateBoundaryEventBehavior in favor of AttachEventBehavior

Closes #1154
Closes #1202
Closes #1204
Closes #1205
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers modeling
3 participants