Skip to content

[7.12] Put environment back in uiFilters (#92561)#92687

Merged
smith merged 2 commits intoelastic:7.12from
smith:backport/7.12/pr-92561
Feb 27, 2021
Merged

[7.12] Put environment back in uiFilters (#92561)#92687
smith merged 2 commits intoelastic:7.12from
smith:backport/7.12/pr-92561

Conversation

@smith
Copy link
Copy Markdown
Contributor

@smith smith commented Feb 24, 2021

Backports the following commits to 7.12:

`environment` is still used in `uiFilters` in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.
# Conflicts:
#	x-pack/plugins/apm/public/context/url_params_context/url_params_context.tsx
@smith smith added the backport This PR is a backport of another PR label Feb 24, 2021
@smith smith enabled auto-merge (squash) February 24, 2021 18:19
@dgieselaar
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 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
apm 5.2MB 5.2MB +31.0B

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

@smith smith merged commit 178481b into elastic:7.12 Feb 27, 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.

3 participants