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.

Sign up for GitHub

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

Jump to bottom

Hanging connection after target removed #1663

Closed
nikku opened this issue May 23, 2022 · 1 comment
Closed

Hanging connection after target removed #1663

nikku opened this issue May 23, 2022 · 1 comment
Assignees
@marstamm
Labels
bug Something isn't working

Comments

@nikku
Copy link
Member

nikku commented May 23, 2022

Describe the Bug

Connections attached to a source or target connection are left hanging when the source or target is removed:

capture 2xTt11_optimized

Steps to Reproduce

  1. Attach association to sequence flow (i.e. via a bpmn:TextAnnotation)
  2. Remove association target
  3. See that association is left hanging

Expected Behavior

Association is removed transitively.

Environment

  • Browser: Any
  • OS: Any
  • Library version: develop
@nikku nikku added bug Something isn't working ready Ready to be worked on labels May 23, 2022
marstamm added a commit to bpmn-io/diagram-js that referenced this issue May 24, 2022
@marstamm
fix(modeling): remove incoming/outgoing on connection.delete
2fc000a 
related to bpmn-io/bpmn-js#1663
@marstamm marstamm mentioned this issue May 24, 2022
Merged
nikku pushed a commit to bpmn-io/diagram-js that referenced this issue May 24, 2022
@marstamm
fix(modeling): remove incoming/outgoing on connection.delete (#648)
40d6f21 
Related to bpmn-io/bpmn-js#1663
@nikku nikku added fixed upstream Requires integration of upstream change and removed ready Ready to be worked on labels May 25, 2022
@nikku
Copy link
Member Author

nikku commented May 25, 2022

Closed via b158add.

@nikku nikku closed this as completed May 25, 2022
@bpmn-io-tasks bpmn-io-tasks bot removed the fixed upstream Requires integration of upstream change label May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marstamm marstamm

Labels
bug Something isn't working
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nikku @marstamm