-
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
Load configuration from EMS-metadata in region-maps #70888
Load configuration from EMS-metadata in region-maps #70888
Conversation
…ad of relying on the vis-state. For EMS-layers, load from EMS-metadata. For self-hosted layers, load from the kibana.yml config.
Pinging @elastic/kibana-gis (Team:Geo) |
@elasticmachine merge upstream |
…nck/kibana into maps/load_config_dynamically
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.
Thanks for the quick diagnosis, fix and minor clean-up. Changes lgtm! Tested locally and was able to repro the error on master and confirm the issue did not exist on this branch.
- code review
- tested locally in chrome and firefox
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.
Changes look great to me as well, thanks for the quick turnaround and collaboration on this. Tested locally on Chrome.
LGTM pending the merge of #70992 and subsequent green CI run 👍
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
I removed v6.8.12 label because it appears this was already in v6.8.11. |
Region-maps would rely on the
visState
to load the human-readable name and attribution of the EMS. While reading out settings of the visState is the canonical approach in Visualize for settings configured in the UX (e.g. metrics and buckets), region maps should not do this for the layer configurations that get rendered to the screen as-is.This PR makes two changes:
maps.regionmap.layers[].attribution
) gets URL-escaped. This parameter is only allowed to be a text-string, as per the existing documentation (https://www.elastic.co/guide/en/kibana/current/settings.html).Blocked by #70992. When that merges, can add appropriate test coverage.