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

[Maps] GeoJson file upload should support geometry-collections #93254

Closed
thomasneirynck opened this issue Mar 2, 2021 · 1 comment · Fixed by #93507
Closed

[Maps] GeoJson file upload should support geometry-collections #93254

thomasneirynck opened this issue Mar 2, 2021 · 1 comment · Fixed by #93507
Assignees
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.13.0 v8.0.0

Comments

@thomasneirynck
Copy link
Contributor

thomasneirynck commented Mar 2, 2021

In 7.12, Maps added support for geometry-collections stored in Elasticsearch.

GeoJson upload does not support GeometryCollections. It breaks in two ways:
- the layer-preview breaks because the label-centroid calculation throws when encountering a geometry-collection
- the upload to elasticsearch fails because the geometries array is not correctly set

To close the loop on this, and remove this odd breakage, GeoJson-upload should support geometry-collections.

@thomasneirynck thomasneirynck added bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v7.13.0 labels Mar 2, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.13.0 v8.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants