search

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

Debounce subscription requests? #392

Open hjoliver opened 4 years ago

hjoliver commented 4 years ago

Whenever data requirements change, a new request is issue to update the merged subscription.

After much discussion (sorry @kinow :grin: ) we have identified at least one scenario where this might need debounce: user quickly clicks on multiple suites in the gscan sidebar, one after the other (potentially with mulitple tabs open in the workflows view too).

Ref #388 and #373

kinow commented 4 years ago

Note to self/later: filtering is normally implemented in JS with debounce. I have not added it as we don't have debounce anywhere in the code (except in the Graph component I think). Leaving it for later if necessary, after discussion, etc