Skip to content

[7.x] [APM] Fix missing RUM url (#42940)#43021

Merged
sorenlouv merged 2 commits intoelastic:7.xfrom
sorenlouv:backport/7.x/pr-42940
Aug 14, 2019
Merged

[7.x] [APM] Fix missing RUM url (#42940)#43021
sorenlouv merged 2 commits intoelastic:7.xfrom
sorenlouv:backport/7.x/pr-42940

Conversation

@sorenlouv
Copy link
Member

Backports the following commits to 7.x:

@sorenlouv sorenlouv added the backport This PR is a backport of another PR label Aug 9, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@sorenlouv sorenlouv force-pushed the backport/7.x/pr-42940 branch from bc05dcf to 0d8f1c2 Compare August 14, 2019 16:22
* [APM] Fix missing RUM url

* Reduce to single conditional branch

* Fix test

* Add tests for `url.full` and `error.page.url`

# Conflicts:
#	x-pack/legacy/plugins/apm/public/components/app/ErrorGroupDetails/DetailView/StickyErrorProperties.test.tsx
#	x-pack/legacy/plugins/apm/public/components/app/ErrorGroupDetails/DetailView/StickyErrorProperties.tsx
@sorenlouv sorenlouv force-pushed the backport/7.x/pr-42940 branch from 0d8f1c2 to 2e1728b Compare August 14, 2019 16:27
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@sorenlouv sorenlouv merged commit f3391bd into elastic:7.x Aug 14, 2019
@sorenlouv sorenlouv deleted the backport/7.x/pr-42940 branch August 14, 2019 20:27
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