Skip to content
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

[8.x] [maps embeddable] fix map layers disappear from map panel when using session storage to continue editing a dashboard (#193629) #193797

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…session storage to continue editing a dashboard (elastic#193629)

Closes elastic#190468 and
elastic#193601

Dashboard unsaved changes for a panel are obtained from the embeddable
comparator subject for each comparator. If comparator subject does not
return a value, then this information is lost when returning to a
dashboard with unsaved changes. This is why layers and filters where
disappearing from a map when returning to a dashboard with unsaved
changes. This PR resolves this issue by providing comparator subjects
and setters for `savedObjectId`, `attributes`, and `mapSettings`.

PR also resolves issue of passing props from `MapRenderer` component
into map embeddable. Instead of passing props via state, props should be
passed from `MapRenderer` parent api.

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit a8ebc7f)
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
maps 1236 1237 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
maps 3.0MB 3.0MB +770.0B

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
maps 28 27 -1

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @nreese

@kibanamachine kibanamachine merged commit 58480de into elastic:8.x Sep 23, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants