-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Refactor AssetLiveDataProvider -> LiveDataProvider (#19453)
## Summary & Motivation The strategy of chunking/batching queries to the server is one I want to reuse on the upcoming asset timeline page. To support that I'm refactoring this class to allow for the core chunking/batching logic to be reused by other queries. ## How I Tested These Changes I relied on the existing jest tests for AssetLiveDataProvider
- Loading branch information
Showing
19 changed files
with
612 additions
and
523 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
f6a0162
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.
Deploy preview for dagit-core-storybook ready!
✅ Preview
https://dagit-core-storybook-df63p3rrj-elementl.vercel.app
Built with commit f6a0162.
This pull request is being automatically deployed with vercel-action