Skip to content

Conversation

@DDDDDanica
Copy link
Contributor

@DDDDDanica DDDDDanica commented Aug 12, 2025

Description

Create Account Component - React Router v6 Migration

Open in GitHub Codespaces

Changelog

CHANGELOG entry: null

Related issues

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/5560

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@DDDDDanica DDDDDanica self-assigned this Aug 12, 2025
@github-actions
Copy link
Contributor

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.

@metamaskbot metamaskbot added the team-extension-platform Extension Platform team label Aug 12, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [5ff9e42]
UI Startup Metrics (1235 ± 72 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1235110315107212911361
load104389913157110911149
domContentLoaded103589413097110801143
domInteractive18146971828
firstPaint688130119341610721152
backgroundConnect2252072629230244
firstReactRender21164252333
getState1258591625
initialActions407610316
loadScripts818689108469864932
setupStore85162912
WebpackHomeuiStartup22451632262319623542567
load16771202197415517491904
domContentLoaded16701196196215417421888
domInteractive181297161468
firstPaint1756635764214301
backgroundConnect3015339333060
firstReactRender21389405116367392
getState3443727814280
initialActions7217217612
loadScripts16651193195015317371877
setupStore256330591734
FirefoxBrowserifyHomeuiStartup13251125176510514011498
load114198513628011981284
domContentLoaded114198413628011971284
domInteractive993430944107204
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect28176893244
firstReactRender24205452338
getState727511633
initialActions30275214
loadScripts112096813478011771267
setupStore8310212627
WebpackHomeuiStartup15531351213912716151785
load12951126164710013691461
domContentLoaded12951125164710013681461
domInteractive89312634392203
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect3020122143356
firstReactRender53418475765
getState10468111039
initialActions30536315
loadScripts1274110216269913481440
setupStore11417819736
Benchmark value 1236 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 225 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 244 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 16 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 2246 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 34 exceeds gate value 29 for chrome webpack home mean getState
Benchmark value 2567 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 68 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 392 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 12 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 3 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 204 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 33 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 14 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 30 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 53 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 203 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 56 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 39 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 15 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: 307ms | Sum of p95 exceeds: 603.8ms
Sum of all benchmark exceeds: 910.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 43 Bytes (0%)
  • ui: 1.12 KiB (0.02%)
  • common: -34 Bytes (0%)

@DDDDDanica DDDDDanica marked this pull request as ready for review August 12, 2025 14:31
Copy link
Contributor

@itsyoboieltr itsyoboieltr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one small thing

const mockStore = configureMockStore([thunk])(mockState);

beforeEach(() => {
jest.clearAllMocks();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesn't this happen automatically as restoreMocks: true is in our jest.config.js? @Gudahtt

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed

  // TODO: enable resetMocks
  // resetMocks: true,

i think that's more what we are looking for

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can try in another PR to remove all the jest.clearAllMocks(); by resolving TODO ~

Copy link
Contributor

@MajorLift MajorLift left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@DDDDDanica DDDDDanica added this pull request to the merge queue Aug 12, 2025
Merged via the queue into main with commit 6198462 Aug 12, 2025
150 of 151 checks passed
@DDDDDanica DDDDDanica deleted the refactor/5560-create-accounr branch August 12, 2025 15:57
@github-actions github-actions bot locked and limited conversation to collaborators Aug 12, 2025
@metamaskbot metamaskbot added the release-13.2.0 Issue or pull request that will be included in release 13.2.0 label Aug 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.2.0 Issue or pull request that will be included in release 13.2.0 size-S team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants