Skip to content

[7.x] [Maps][Vega] Tidy up mapbox-gl imports (#99733)#99920

Merged
thomasneirynck merged 1 commit intoelastic:7.xfrom
thomasneirynck:backport/7.x/pr-99733
May 12, 2021
Merged

[7.x] [Maps][Vega] Tidy up mapbox-gl imports (#99733)#99920
thomasneirynck merged 1 commit intoelastic:7.xfrom
thomasneirynck:backport/7.x/pr-99733

Conversation

@thomasneirynck
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

VegaMaps now instantiates mapbox-gl the same way as the Maps app.
@thomasneirynck thomasneirynck added the backport This PR is a backport of another PR label May 12, 2021
@thomasneirynck thomasneirynck enabled auto-merge (squash) May 12, 2021 13:37
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
visTypeVega 241 243 +2

Async chunks

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

id before after diff
visTypeVega 2.6MB 2.6MB -71.1KB
Unknown metric groups

miscellaneous assets size

id before after diff
visTypeVega 0.0B 550.3KB ⚠️ +550.3KB

References to deprecated APIs

id before after diff
canvas 29 25 -4
crossClusterReplication 8 6 -2
fleet 4 2 -2
globalSearch 4 2 -2
indexManagement 12 7 -5
infra 5 3 -2
licensing 18 15 -3
monitoring 109 56 -53
total -73

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

@thomasneirynck thomasneirynck merged commit 4e75493 into elastic:7.x May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants