-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: pass abort signal to cancel api request on query-key change or … #4193
Conversation
509f112
to
1df8fae
Compare
1056a9b
to
d2dadae
Compare
075a074
to
0831e34
Compare
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.
I have tried to run this pr locally but it doesn't seem to cancel the requests. Will share the exact dashboard
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.
LGTM, we can create PR's for other pages as well.
0831e34
to
c477d03
Compare
7be73a7
to
066e8bf
Compare
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.
functionality is working as expected ... though small comments
…dashboard unmount
Summary
Pass abort signal to cancel API request on query-key change or dashboard unmount
Related Issues / PR's
Fixes #4147
Before:
https://www.loom.com/share/a10502d695be4fd5ab40140ca136d015?sid=0509f83f-33ab-40a0-9cba-288af24b5193
After:
https://www.loom.com/share/1914371759ea4f24807813d83eee0367?sid=6d5a1658-8e27-4fcc-9043-4cb65d38c9c3)
Affected Areas and Manually Tested Areas
Dashboard Page