-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Lens] Move dataViews cache into main Lens state #137309
Merged
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
57d814d
Move lens dataViews state into main state
dej611 751c2a2
:fire: Remove some old cruft from the code
dej611 c7a9ce8
:bug: Fix dataViews layer change
dej611 79566af
:bug: Fix datasourceLayers refs
dej611 4822b36
:fire: Remove more old cruft
dej611 220aa39
:bug: Fix bug when loading SO
dej611 759f19a
:bug: Fix initial existence flag
dej611 7f590da
:label: Fix type issues
dej611 12b91fc
:label: Fix types and tests
dej611 2ee6f6d
:label: Fix types issues
dej611 ce7bddd
:white_check_mark: Fix more tests
dej611 80647be
:white_check_mark: Fix with new dataViews structure
dej611 63bf020
:white_check_mark: Fix more test mocks
dej611 3a7f9fb
:white_check_mark: More tests fixed
dej611 2fc1f0f
:fire: Removed unused prop
dej611 e9abe5a
:white_check_mark: Down to single broken test suite
dej611 ca51c4f
Merge remote-tracking branch 'upstream/main' into feature/dataview-state
dej611 13a669e
:label: Fix type issue
dej611 120e52a
Merge branch 'main' into feature/dataview-state
flash1293 58b873a
:ok_hand: Integrate selector feedback
dej611 70cab64
:white_check_mark: Fix remaining unit tests
dej611 2301d19
Merge remote-tracking branch 'upstream/main' into feature/dataview-state
dej611 e68a013
:label: fix type issues
dej611 d594e09
:bug: Fix bug when creating dataview in place
dej611 d32f9dd
Merge remote-tracking branch 'upstream/main' into feature/dataview-state
dej611 5a5d3b9
:bug: Fix edit + remove field flow
dej611 0ed3fdf
Update x-pack/plugins/lens/public/visualizations/xy/types.ts
dej611 07e3136
:camera_flash: Fix snapshot
dej611 d317513
:bug: Fix the dataViews switch bug
dej611 8510724
Merge remote-tracking branch 'upstream/main' into feature/dataview-state
dej611 1b64dda
:fire: remove old cruft
dej611 958ae77
Merge remote-tracking branch 'upstream/main' into feature/dataview-state
dej611 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
8 changes: 8 additions & 0 deletions
8
x-pack/plugins/lens/public/app_plugin/__snapshots__/app.test.tsx.snap
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
When a new data view is created from within Lens, the data panel crashes (seems like the new data view isn't added to the cache correctly?)