equinor / flyt

Equinor Flyt (aka. VSM - Value Stream Mapping)
https://flyt.equinor.com
MIT License
4 stars 1 forks source link

Moving a choice that's inside a loop raises an error #687

Closed nesadrian closed 2 months ago

nesadrian commented 4 months ago

To Reproduce Steps to reproduce the behavior:

  1. Go to a process
  2. Add a choice card
  3. Add a sub activity card beneath one of the choices children
  4. Merge the new sub activity with the choice card
  5. Try to move the choice card
  6. See error

Expected behavior The choice should be movable even though it's in a loop.

Screenshots image

QiJin-Bouvet commented 4 months ago

688 is a duplicate of this issue.

Will test both scenarios when this issue is fixed.

nesadrian commented 4 months ago

@QiJin-Bouvet Ready for test in POC DEV/QA. I discovered a new issue while working on this. If you move a choice that's inside a loop to another main activity column then it will still have the merge connection to the top of the loop. I will create a new issue for this.

QiJin-Bouvet commented 4 months ago

Tested in qa-graph-poc. The choice card in loop is movable. But still merge to the previous tree. This will be fixed in another card #690

nesadrian commented 4 months ago

@QiJin-Bouvet A new. issue has been made here #690