Skip to content

[8.8] [maps] fix Map orphans sources on layer deletion (#159067) - #159160

Merged
kibanamachine merged 1 commit into
elastic:8.8from
kibanamachine:backport/8.8/pr-159067
Jun 6, 2023
Merged

kibanamachine merged 1 commit into
elastic:8.8from
kibanamachine:backport/8.8/pr-159067

Conversation

@kibanamachine

Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.8:

Questions ?

Please refer to the Backport tool documentation

Fixes elastic#158133

Sources where not getting removed because `return` was used instead of
`continue` in `for...in` loop. This caused the function to return
instead of processing remaining sources.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit fcd1950)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 6, 2023
@kibanamachine
kibanamachine enabled auto-merge (squash) June 6, 2023 20:43
@kibana-ci

Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
maps 2.8MB 2.8MB -77.0B
Unknown metric groups

API count

id before after diff
@kbn/mapbox-gl 534 535 +1

ESLint disabled in files

id before after diff
maps 26 27 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 17 19 +2
securitySolution 400 404 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 18 20 +2
maps 74 75 +1
securitySolution 480 484 +4
total +7

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

cc @nreese

@kibanamachine
kibanamachine merged commit 64b1138 into elastic:8.8 Jun 6, 2023
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.

3 participants