Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

web: migrate embedded web app to react-router 6 API - #47452

Merged
valerybugakov merged 2 commits into
mainfrom
vb/react-router-migration-1
Feb 8, 2023
Merged

web: migrate embedded web app to react-router 6 API#47452
valerybugakov merged 2 commits into
mainfrom
vb/react-router-migration-1

Conversation

@valerybugakov

@valerybugakov valerybugakov commented Feb 8, 2023

Copy link
Copy Markdown
Member

Context

Test plan

CI

App preview:

Check out the client app preview documentation to learn more.

@valerybugakov valerybugakov self-assigned this Feb 8, 2023
@cla-bot cla-bot Bot added the cla-signed label Feb 8, 2023
@valerybugakov
valerybugakov requested review from a team and vovakulikov February 8, 2023 07:21
@github-actions github-actions Bot added the team/code-exploration Issues owned by the Code Exploration team label Feb 8, 2023
@sg-e2e-regression-test-bob

sg-e2e-regression-test-bob commented Feb 8, 2023

Copy link
Copy Markdown

Bundle size report 📦

Initial size Total size Async size Modules
-0.11% (-3.02 kb) -0.02% (-2.99 kb) 0.00% (+0.03 kb) 0.00% (0)

Look at the Statoscope report for a full comparison between the commits 8305722 and a0745f9 or learn more.

Open explanation
  • Initial size is the size of the initial bundle (the one that is loaded when you open the page)
  • Total size is the size of the initial bundle + all the async loaded chunks
  • Async size is the size of all the async loaded chunks
  • Modules is the number of modules in the initial bundle

@valerybugakov
valerybugakov enabled auto-merge (squash) February 8, 2023 07:57
@valerybugakov
valerybugakov merged commit 47b6ae8 into main Feb 8, 2023
@valerybugakov
valerybugakov deleted the vb/react-router-migration-1 branch February 8, 2023 08:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed storm team/code-exploration Issues owned by the Code Exploration team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants