Skip to content

fix(dashboards): Check query ID before moving out of loading state#106449

Merged
narsaynorath merged 1 commit into
masterfrom
nar/fix/dashboards-check-query-id-before-changing-loading-false
Jan 16, 2026
Merged

fix(dashboards): Check query ID before moving out of loading state#106449
narsaynorath merged 1 commit into
masterfrom
nar/fix/dashboards-check-query-id-before-changing-loading-false

Conversation

@narsaynorath

@narsaynorath narsaynorath commented Jan 16, 2026

Copy link
Copy Markdown
Member

Two requests can clobber each other when another request is fired before the previous one finishes. This checks that the current query ID we've generated is currently being processed, ignoring past requests for state changes (i.e. we don't change loading -> false if it's not the most current request)

Fixes EXP-712

@narsaynorath narsaynorath requested a review from a team as a code owner January 16, 2026 17:11
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 16, 2026
@linear

linear Bot commented Jan 16, 2026

Copy link
Copy Markdown

@narsaynorath narsaynorath enabled auto-merge (squash) January 16, 2026 17:17
@narsaynorath narsaynorath merged commit 782e85e into master Jan 16, 2026
56 checks passed
@narsaynorath narsaynorath deleted the nar/fix/dashboards-check-query-id-before-changing-loading-false branch January 16, 2026 17:19
BYK pushed a commit that referenced this pull request Jan 19, 2026
…106449)

Two requests can clobber each other when another request is fired before
the previous one finishes. This checks that the current query ID we've
generated is currently being processed, ignoring past requests for state
changes (i.e. we don't change loading -> `false` if it's not the most
current request)

Fixes EXP-712
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants