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] Fix filters reappearing in the saved object when saving #110460
Conversation
Pinging @elastic/kibana-app (Team:KibanaApp) |
@@ -384,51 +348,5 @@ export function getLastKnownDocWithoutPinnedFilters(doc?: Document) { | |||
: doc; | |||
} | |||
|
|||
export const getPersistedDoc = async ({ |
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.
this piece is moved to another file.
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.
Code LGTM, I tested it locally in Safari and works fine. I tested various combinations and all of them work fine :)
3766647
to
429a7e9
Compare
💚 Build SucceededMetrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
…ic#110460) * bugs fixed * test for loading * fix workspace panel * Update x-pack/plugins/lens/public/xy_visualization/visualization.tsx * revert useEffect for external embeddables
…ic#110460) * bugs fixed * test for loading * fix workspace panel * Update x-pack/plugins/lens/public/xy_visualization/visualization.tsx * revert useEffect for external embeddables
…) (#110590) * bugs fixed * test for loading * fix workspace panel * Update x-pack/plugins/lens/public/xy_visualization/visualization.tsx * revert useEffect for external embeddables Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com>
…) (#110589) * bugs fixed * test for loading * fix workspace panel * Update x-pack/plugins/lens/public/xy_visualization/visualization.tsx * revert useEffect for external embeddables Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com>
Summary
Fixes #110248
Also fixes the issue with loading empty document from the search bar. To reproduce, load existing document and then search for Lens new document in the top search bar - the bug was caused by not properly cleaning initial state for the empty state.
Checklist