-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Refetch indexeddb cached query immediately #22517
Conversation
Deploy preview for dagit-core-storybook ready! ✅ Preview Built with commit cbc7a83. |
}, [key, version, getCachedData]); | ||
|
||
React.useEffect(() => { | ||
fetch(true); |
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.
The true argument here makes the fetch
call skip the cache.
## Summary & Motivation Recent changes to this file regressed the immediate fetch from the server in favor of only fetching from cache. This PR adds back the fetching from the server behavior to update the cache. ## How I Tested These Changes App proxy, see requests fire in the network panel: <img width="682" alt="Screenshot 2024-06-12 at 7 51 09 PM" src="https://github.com/dagster-io/dagster/assets/2286579/f1301542-e542-4cd8-96cd-262776671e65">
## Summary & Motivation Recent changes to this file regressed the immediate fetch from the server in favor of only fetching from cache. This PR adds back the fetching from the server behavior to update the cache. ## How I Tested These Changes App proxy, see requests fire in the network panel: <img width="682" alt="Screenshot 2024-06-12 at 7 51 09 PM" src="https://github.com/dagster-io/dagster/assets/2286579/f1301542-e542-4cd8-96cd-262776671e65">
## Summary & Motivation Recent changes to this file regressed the immediate fetch from the server in favor of only fetching from cache. This PR adds back the fetching from the server behavior to update the cache. ## How I Tested These Changes App proxy, see requests fire in the network panel: <img width="682" alt="Screenshot 2024-06-12 at 7 51 09 PM" src="https://github.com/dagster-io/dagster/assets/2286579/f1301542-e542-4cd8-96cd-262776671e65">
Summary & Motivation
Recent changes to this file regressed the immediate fetch from the server in favor of only fetching from cache. This PR adds back the fetching from the server behavior to update the cache.
How I Tested These Changes
App proxy, see requests fire in the network panel: