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

DMN - Annotation flow disappears after renaming decision in table editor #2052

Closed
azeghers opened this issue Jan 6, 2021 · 4 comments · Fixed by #2082
Closed

DMN - Annotation flow disappears after renaming decision in table editor #2052

azeghers opened this issue Jan 6, 2021 · 4 comments · Fixed by #2082
Assignees
Labels
bug Something isn't working spring cleaning Could be cleaned up one day
Milestone

Comments

@azeghers
Copy link
Contributor

azeghers commented Jan 6, 2021

Describe the Bug

annotation-2021-01-06_14.48.59.mp4

Here are the logs after saving and reopening the diagram:

image

Steps to Reproduce

-Create Decision
-Name it
-Add Annotation
-Morph into Decision Table
-Rename inside of Decision Table mode
-Go back to DRD
-See that the Annotation is unlinked

Expected Behavior

The annotation stays linked to the decision element

Environment

  • OS: Ubuntu 20.04
  • Camunda Modeler Version: v4.5.0-rc.0

Related to #1874

@azeghers azeghers added the bug Something isn't working label Jan 6, 2021
@barmac
Copy link
Collaborator

barmac commented Jan 6, 2021

Related to #1814 and #1763

I can also reproduce that in 4.4.0

@barmac barmac added the spring cleaning Could be cleaned up one day label Jan 6, 2021
@pinussilvestrus
Copy link
Contributor

Related to #1874.

@MaxTru MaxTru added the backlog Queued in backlog label Jan 7, 2021 — with bpmn-io-tasks
@philippfromme philippfromme self-assigned this Jan 18, 2021
@philippfromme philippfromme added in progress Currently worked on and removed backlog Queued in backlog labels Jan 18, 2021 — with bpmn-io-tasks
@philippfromme
Copy link
Contributor

philippfromme commented Jan 18, 2021

The issue is that when replacing the decision the association's parent from the model's perspective is changed to the text annotation. In the case of information requirements and such the parent is indeed the target element. Not in this case since the association is an artifact.

This issue was fixed before (bpmn-io/dmn-js@90a7952#diff-a42b136569b62592b63126d0bfe523df9cfd77884dfd10741252ceead428e185R265) but then broken again (bpmn-io/dmn-js@853eea1).

So we're fixing it again. ♻️

@philippfromme philippfromme added the fixed upstream Requires integration of upstream change label Jan 19, 2021
@nikku
Copy link
Member

nikku commented Jan 19, 2021

Fixed upstream (again) via bpmn-io/dmn-js#620.

@nikku nikku added this to the M45 milestone Jan 19, 2021
@nikku nikku removed the in progress Currently worked on label Jan 19, 2021
nikku added a commit that referenced this issue Jan 28, 2021
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed fixed upstream Requires integration of upstream change labels Jan 28, 2021
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jan 29, 2021
nikku added a commit that referenced this issue Jan 29, 2021
MaxTru pushed a commit that referenced this issue Feb 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working spring cleaning Could be cleaned up one day
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants