Fix N-window selector sending mutation upon navigation to stopped workflows - Githubissues

cylc / cylc-ui

Web app for monitoring and controlling Cylc workflows

https://cylc.github.io

GNU General Public License v3.0

37 stars 27 forks source link

Fix N-window selector sending mutation upon navigation to stopped workflows #1786

Closed MetRonnie closed 6 months ago

MetRonnie commented 6 months ago

Fixes an unreleased bug on master. Also adds some tests. Follow-up to #1755

How to reproduce the bug on master

Start a workflow and use the UI to set the n-window to a value other than 1, e.g. 2
Stop the workflow
Navigate to the dashboard or a different workflow
Navigate back to the original workflow
You should see an error snackbar for a failed mutation

Check List

[x] I have read CONTRIBUTING.md and added my name as a Code Contributor.
[x] Contains logically grouped changes (else tidy your branch by rebase).
[x] Does not contain off-topic changes (use other PRs for other changes).
[x] Tests are included
[x] No changelog entry needed as fixes unreleased bug