-
Notifications
You must be signed in to change notification settings - Fork 5.3k
refactor(5557): migrate onboarding/ to react-router-v5-compat #35005
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
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
✨ Files requiring CODEOWNER review ✨🔐 @MetaMask/web3auth (29 files, +410 -298)
|
2ecb43e
to
0eedfce
Compare
0eedfce
to
c08b322
Compare
Builds ready [c08b322]
UI Startup Metrics (1257 ± 88 ms)
Benchmark value 227 exceeds gate value 10 for chrome browserify home mean backgroundConnect Benchmark value 7 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 832 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 1458 exceeds gate value 1365 for chrome browserify home p95 uiStartup Benchmark value 1249 exceeds gate value 1190 for chrome browserify home p95 load Benchmark value 1241 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 241 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 28 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 997 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 2501 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 1881 exceeds gate value 1711 for chrome webpack home mean load Benchmark value 1874 exceeds gate value 1704 for chrome webpack home mean domContentLoaded Benchmark value 204 exceeds gate value 199 for chrome webpack home mean firstPaint Benchmark value 8 exceeds gate value 7 for chrome webpack home mean initialActions Benchmark value 1869 exceeds gate value 1699 for chrome webpack home mean loadScripts Benchmark value 35 exceeds gate value 32 for chrome webpack home mean setupStore Benchmark value 2896 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 2187 exceeds gate value 2030 for chrome webpack home p95 load Benchmark value 2180 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded Benchmark value 390 exceeds gate value 334 for chrome webpack home p95 firstPaint Benchmark value 403 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 257 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 12 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 2166 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 296 exceeds gate value 65 for chrome webpack home p95 setupStore Benchmark value 30 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 3 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 10 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 212 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 34 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 9 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 34 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 30 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 55 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 5 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 254 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 51 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 65 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 49 exceeds gate value 32 for firefox webpack home p95 getState Benchmark value 15 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 1109ms | Sum of p95 exceeds: 2062.8ms Sum of all benchmark exceeds: 3171.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
8e8a163
to
2dc1083
Compare
2dc1083
to
f19fc35
Compare
❌ test-e2e-chrome-api-specs failed. View the html report here. |
Builds ready [f19fc35]
UI Startup Metrics (1181 ± 63 ms)
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 233 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 18 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 2235 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 30 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 2551 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 66 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 395 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 280 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 9 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 29 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 5 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 211 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 26 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 11 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 32 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 55 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 5 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 249 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 51 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 65 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 13 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 291ms | Sum of p95 exceeds: 597.8ms Sum of all benchmark exceeds: 888.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [492ad5e]
UI Startup Metrics (1197 ± 66 ms)
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 240 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 17 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 35 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 33 exceeds gate value 32 for chrome webpack home mean setupStore Benchmark value 2491 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 64 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 381 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 280 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 13 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 289 exceeds gate value 65 for chrome webpack home p95 setupStore Benchmark value 31 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 28 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 3 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 209 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 40 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 12 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 33 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1658 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 103 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 33 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 58 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 4 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 262 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 61 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 70 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 13 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 36 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 311ms | Sum of p95 exceeds: 810.8ms Sum of all benchmark exceeds: 1121.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
492ad5e
to
90146fd
Compare
Builds ready [95741f3]
UI Startup Metrics (1219 ± 67 ms)
Benchmark value 28 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 17 exceeds gate value 15 for chrome browserify home mean getState Benchmark value 7 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 1183 exceeds gate value 1180 for chrome browserify home p95 firstPaint Benchmark value 253 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 51 exceeds gate value 45 for chrome browserify home p95 firstReactRender Benchmark value 24 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 951 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 43 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 2457 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 284 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 10 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 114 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 33 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 27 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 5 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 246 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 11 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 32 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 109 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 33 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 43 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 3 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 271 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 8 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 298ms | Sum of p95 exceeds: 558.8ms Sum of all benchmark exceeds: 856.8ms |
00c8214
📊 Page Load Benchmark ResultsCurrent Commit: 📄 https://metamask.github.io/test-dapp/Samples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [00c8214]
UI Startup Metrics (1209 ± 56 ms)
Benchmark value 25 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 254 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 11 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 34 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 8 exceeds gate value 7 for chrome webpack home mean initialActions Benchmark value 2533 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 75 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 280 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 11 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 32 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 27 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 7 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 223 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 7 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 45 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 107 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 34 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 43 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 3 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 281 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 51 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 6 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 286ms | Sum of p95 exceeds: 613.8ms Sum of all benchmark exceeds: 899.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 https://metamask.github.io/test-dapp/Samples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [ccfd9ac]
UI Startup Metrics (1257 ± 66 ms)
Benchmark value 1090 exceeds gate value 1070 for chrome browserify home mean load Benchmark value 1081 exceeds gate value 1061 for chrome browserify home mean domContentLoaded Benchmark value 252 exceeds gate value 10 for chrome browserify home mean backgroundConnect Benchmark value 26 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 838 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 1181 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 271 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 14 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 944 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 18 exceeds gate value 17 for chrome browserify home p95 setupStore Benchmark value 38 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 9 exceeds gate value 7 for chrome webpack home mean initialActions Benchmark value 2469 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 278 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 12 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 1456 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1261 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1261 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 123 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 35 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 29 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 1234 exceeds gate value 1230 for firefox browserify home mean loadScripts Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 282 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 79 exceeds gate value 70 for firefox browserify home p95 backgroundConnect Benchmark value 16 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 37 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 102 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 33 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 43 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 282 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 54 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 13 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 478ms | Sum of p95 exceeds: 635.8ms Sum of all benchmark exceeds: 1113.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
📊 Page Load Benchmark ResultsCurrent Commit: 📄 https://metamask.github.io/test-dapp/Samples: 100 Summary
📈 Detailed Results
Results generated automatically by MetaMask CI |
Builds ready [b10df64]
UI Startup Metrics (1219 ± 60 ms)
Benchmark value 24 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 6 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 1182 exceeds gate value 1180 for chrome browserify home p95 firstPaint Benchmark value 266 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 14 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 35 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 2561 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 281 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 10 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 1795 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1558 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1557 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 142 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 54 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 33 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 1515 exceeds gate value 1230 for firefox browserify home mean loadScripts Benchmark value 16 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 2139 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 1821 exceeds gate value 1495 for firefox browserify home p95 load Benchmark value 1821 exceeds gate value 1495 for firefox browserify home p95 domContentLoaded Benchmark value 343 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 150 exceeds gate value 70 for firefox browserify home p95 backgroundConnect Benchmark value 14 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 1793 exceeds gate value 1475 for firefox browserify home p95 loadScripts Benchmark value 67 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1678 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1420 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1420 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 113 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 34 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 49 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 7 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1395 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 1938 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 238 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 58 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 6 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 1852ms | Sum of p95 exceeds: 2284.8ms Sum of all benchmark exceeds: 4136.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description
Migrate onboarding flow components to React Router v6 compatibility.
Changelog
CHANGELOG entry: null
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/5557
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist