Skip to content

[7.x] Update to latest rum-react (#78193)#78517

Merged
smith merged 3 commits intoelastic:7.xfrom
smith:backport/7.x/pr-78193
Sep 28, 2020
Merged

[7.x] Update to latest rum-react (#78193)#78517
smith merged 3 commits intoelastic:7.xfrom
smith:backport/7.x/pr-78193

Conversation

@smith
Copy link
Copy Markdown
Contributor

@smith smith commented Sep 24, 2020

Backports the following commits to 7.x:

* Update to latest rum-react

The latest version fixes a problem where you would get a bunch of warnings that you couldn't turn off if you used `render` instead of component with a route. This was causing us to use `component` in some places where `render` should be used.

The latest version fixes this problem so we change back to `render`
where appropriate.

Also make our `ApmRoute` a `Route` instead of `any`.
# Conflicts:
#	x-pack/package.json
#	x-pack/plugins/apm/typings/apm_rum_react.d.ts
#	yarn.lock
@smith smith added the backport This PR is a backport of another PR label Sep 24, 2020
@smith
Copy link
Copy Markdown
Contributor Author

smith commented Sep 28, 2020

@elasticmachine merge upstream

@smith
Copy link
Copy Markdown
Contributor Author

smith commented Sep 28, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@smith smith merged commit 903bebe into elastic:7.x Sep 28, 2020
@smith smith deleted the backport/7.x/pr-78193 branch September 28, 2020 16:01
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