-
Notifications
You must be signed in to change notification settings - Fork 15
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.
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
VizPanel: Allow queries to be cancelled #220
Conversation
Would appreciate a demo scene in scene-app for testing this behavior :) |
I think the lazy loading demo should work for testing (the queries take at least 5 seconds) |
Not sure what the best approach is for checking if a SQR is shared between multiple panels |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now @kaydelaney! I see only one comment left from @torkelo and I think we should be good to land this :)
🚀 PR was released in |
Not sure this is the right approach (specifically with how I'm getting the closest SceneQueryRunner), but it seems to work.
Part of #207
Will tackle query variable cancellation in another PR
📦 Published PR as canary version:
0.17.0--canary.220.5309552119.0
✨ Test out this PR locally via:
npm install @grafana/scenes@0.17.0--canary.220.5309552119.0 # or yarn add @grafana/scenes@0.17.0--canary.220.5309552119.0