Flow: cancelling a second change in another check, toggles the first change
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
Repro steps
- Have a flow configuration with at least two checks/conditions
- Change the first condition and click cancel
- Change the second condition and click cancel
Expected
Only the relevant changes are undone
Actual
The first condition toggles. Cf this video:
https://github.com/user-attachments/assets/29d3d5ef-f822-4265-990c-e7c2e67c3c85
Notes
I spent hours debugging it, but the only take away i have is that something is out of sync and keeps overwriting itself. I am suspecting the check in Check.vue, but perhaps something else. My head is spinning.
Found while working at https://github.com/nextcloud/server/pull/50783
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the flow configuration UI and the suspected Check.vue component. Reproduce the two-condition sequence: change and cancel the first condition, then change and cancel the second. Done means cancelling each condition only undoes that condition, without toggling the first one.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100