Skip to content

[7.x] [Canvas] Switch Canvas to use React Router (#100579)#101659

Merged
crob611 merged 1 commit intoelastic:7.xfrom
crob611:backport/7.x/pr-100579
Jun 8, 2021
Merged

[7.x] [Canvas] Switch Canvas to use React Router (#100579)#101659
crob611 merged 1 commit intoelastic:7.xfrom
crob611:backport/7.x/pr-100579

Conversation

@crob611
Copy link
Contributor

@crob611 crob611 commented Jun 8, 2021

Backports the following commits to 7.x:

* Switch Canvas to use React Router

* Fix typescript errors

* Remove @scant/router from package.json

* Fix tests

* Fix functional test

* Fix functional tests

* Fix bad merge in package.json

* Cleanup from code review comments

* Fix double basepath append

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
# Conflicts:
#	package.json
@crob611 crob611 added the backport This PR is a backport of another PR label Jun 8, 2021
@crob611 crob611 enabled auto-merge (squash) June 8, 2021 18:38
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
canvas 1099 1089 -10

Async chunks

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

id before after diff
canvas 1.3MB 1.3MB -22.8KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
canvas 532.8KB 533.4KB +575.0B

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

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