-
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
[Dashboard] Move Sharing Saved Object Props into Meta Object #119913
[Dashboard] Move Sharing Saved Object Props into Meta Object #119913
Conversation
Pinging @elastic/kibana-gis (Team:Geo) |
Pinging @elastic/kibana-presentation (Team:Presentation) |
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
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.
maps changes LGTM - thanks for fixing
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.
AppService code lgtm
src/plugins/embeddable/public/lib/attribute_service/attribute_service.tsx
Outdated
Show resolved
Hide resolved
…service.tsx Co-authored-by: Anton Dosov <dosantappdev@gmail.com>
@elasticmachine merge upstream |
make sure conflict error logic will execute
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.
tested and works fine for Lens 🆗 Thanks a lot for fixing this!
💚 Build Succeeded
Metrics [docs]Async chunks
Public APIs missing exports
Page load bundle
History
To update your PR or re-run it, just comment with: |
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.
Presentation changes look good 👍
…#119913) Co-authored-by: Anton Dosov <dosantappdev@gmail.com> Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com> # Conflicts: # x-pack/plugins/lens/public/embeddable/embeddable.tsx
…#119913) Co-authored-by: Anton Dosov <dosantappdev@gmail.com> Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com> # Conflicts: # x-pack/plugins/lens/public/embeddable/embeddable.tsx
#120176) Co-authored-by: Anton Dosov <dosantappdev@gmail.com> Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com> # Conflicts: # x-pack/plugins/lens/public/embeddable/embeddable.tsx
#120177) Co-authored-by: Anton Dosov <dosantappdev@gmail.com> Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com> # Conflicts: # x-pack/plugins/lens/public/embeddable/embeddable.tsx
…#119913) Co-authored-by: Anton Dosov <dosantappdev@gmail.com> Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com>
…#119913) Co-authored-by: Anton Dosov <dosantappdev@gmail.com> Co-authored-by: Marta Bondyra <marta.bondyra@gmail.com>
Summary
Fixes #119677.
This PR makes the attribute service's unwrap method return a generically typed meta object alongside the saved object attributes. Getting the input as a value type will not include this, but when using the unwrap method to load the saved object for a by reference panel, the unwrap meta info can be used.
The
SharingSavedObjectProps
has been moved into this new unwrap meta key for both Maps and Lens.Checklist
Delete any items that are not applicable to this PR.