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

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

Merged
merged 2 commits into from
Aug 14, 2019

Conversation

sorenlouv
Copy link
Member

Backports the following commits to 7.x:

@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants