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.

Sign up for GitHub

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

Jump to bottom

graphql subs initial burst fix #5225

Merged

oliver-sanders merged 1 commit into cylc:master from dwsutherland:subs-initial-burst-fix

Nov 14, 2022

Member

dwsutherland commented Nov 14, 2022

closes cylc/cylc-uiserver#384

Even though this is a fix, the bug has no impact on the current UI.

Check List

I have read CONTRIBUTING.md and added my name as a Code Contributor.
Contains logically grouped changes (else tidy your branch by rebase).
Does not contain off-topic changes (use other PRs for other changes).
Tests aren't included, we don't have subscriptions at the scheduler yet.
CHANGES.md entry included if this is a change that can affect users
Cylc-Doc pull request opened if required at cylc/cylc-doc/pull/XXXX.
If this is a bug fix, PR should be raised against the relevant ?.?.x branch.


          graphql subs initial burst fix

71683f7

dwsutherland added the small label

dwsutherland requested a review from oliver-sanders

November 14, 2022 08:52

dwsutherland self-assigned this

oliver-sanders added this to the cylc-8.1.0 milestone

oliver-sanders approved these changes

View reviewed changes

Member

oliver-sanders left a comment

Confirmed this fixes the issue, thanks David!

oliver-sanders merged commit 82d4d67 into cylc:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels