Skip to content

Conversation

@gantunesr
Copy link
Member

@gantunesr gantunesr commented Mar 28, 2025

Description

Open in GitHub Codespaces

This ticket refactors the logic to remove EVM networks from the network picker

Related issues

Fixes: https://github.com/MetaMask/accounts-planning/issues/868

Manual testing steps

  1. Select an EVM network that is not Ethereum, e.g., Linea, Polygon, BSC
  2. Change the selected network to Solana
  3. Delete the previous EVM selected network
  4. The network should be removed and when selecting an EVM account, the network should automatically change to Ethereum mainnet

Screenshots/Recordings

Not applicable

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.

@metamaskbot metamaskbot added the team-accounts-framework Accounts Framework team label Apr 17, 2025
@gantunesr gantunesr marked this pull request as ready for review April 17, 2025 01:54
@gantunesr gantunesr added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Apr 17, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [7be5540]
UI Startup Metrics (1211 ± 74 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1211106816247412551321
load104892813476810741173
domContentLoaded104190213376810691164
domInteractive18136291729
firstPaint68183119342410621152
backgroundConnect84859815
firstReactRender20144352133
getState145171181828
initialActions001001
loadScripts805658106365832919
setupStore75152813
WebpackHomeuiStartup20821696255317622192332
load16101320195412717131821
domContentLoaded16041316193112517071816
domInteractive15115581337
firstPaint1896356676239325
backgroundConnect241068122753
firstReactRender20455401123340354
getState1034461222
initialActions316135
loadScripts15991314190712417041803
setupStore216319382031
FirefoxBrowserifyHomeuiStartup13821190167111714541597
load12271033151712013091448
domContentLoaded12271033151712013091448
domInteractive1023724434119162
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2312106142344
firstReactRender24195452430
getState84436818
initialActions001001
loadScripts12071018149412112881432
setupStore6419268
WebpackHomeuiStartup16171376221616917042011
load13791174198516014491735
domContentLoaded13791173198416014491735
domInteractive903638341100156
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2515105152442
firstReactRender37316453946
getState1043351024
initialActions102111
loadScripts13561154188415514291716
setupStore105508923
cc: @HowardBraham
Benchmark value 1618 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 2011 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1735 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1735 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 1716 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 3ms | Sum of p95 exceeds: 312ms
Sum of all benchmark exceeds: 315ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 142 Bytes (0%)
  • ui: -417 Bytes (-0.01%)
  • common: 0 Bytes (0%)

ccharly
ccharly previously approved these changes Apr 22, 2025
Copy link
Contributor

@ccharly ccharly left a comment

Choose a reason for hiding this comment

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

LGTM + tested, just a left nit suggestion.

…-delete-network.component.js

Co-authored-by: Charly Chevalier <charly.chevalier@consensys.net>
…-delete-network.component.js

Co-authored-by: Charly Chevalier <charly.chevalier@consensys.net>
ccharly
ccharly previously approved these changes Apr 22, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [3b432a0]
UI Startup Metrics (1244 ± 64 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1244109814466412711384
load107396012556011021193
domContentLoaded106695512486010941184
domInteractive18146671729
firstPaint592137125542510661153
backgroundConnect84284814
firstReactRender22165062339
getState1454092033
initialActions001001
loadScripts82672999656853932
setupStore85142813
WebpackHomeuiStartup21851749266917623152409
load16861371207313317721872
domContentLoaded16781367206113117661861
domInteractive16125891440
firstPaint1896950369225329
backgroundConnect271087163066
firstReactRender21655370119337360
getState154197211632
initialActions317146
loadScripts16741366203713017621853
setupStore267293462244
FirefoxBrowserifyHomeuiStartup13471178182711014011570
load12041049170411612671446
domContentLoaded12041049170311612661446
domInteractive1043725733118162
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2312222251944
firstReactRender22195152226
getState74263810
initialActions001001
loadScripts11831027168111912491431
setupStore5424366
WebpackHomeuiStartup14991327195012415931725
load12801129168111213631473
domContentLoaded12801129168111213631473
domInteractive84473103289143
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21155762233
firstReactRender35294843644
getState84335920
initialActions002111
loadScripts12611114166111213431453
setupStore105689831
cc: @HowardBraham
Benchmark value 1244 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1074 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1067 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 1384 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1194 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1184 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 31 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 20ms | Sum of p95 exceeds: 30ms
Sum of all benchmark exceeds: 50ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 142 Bytes (0%)
  • ui: -417 Bytes (-0.01%)
  • common: 0 Bytes (0%)

owencraston
owencraston previously approved these changes Apr 22, 2025
@gantunesr gantunesr enabled auto-merge April 22, 2025 23:13
@gantunesr gantunesr added this pull request to the merge queue Apr 23, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [97de83d]
UI Startup Metrics (1210 ± 75 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1210107615747512471323
load104795413626810821160
domContentLoaded104194813516710741153
domInteractive17136271628
firstPaint70879136242210591160
backgroundConnect74172710
firstReactRender20164852131
getState1353581927
initialActions001001
loadScripts804702107761834913
setupStore85193915
WebpackHomeuiStartup20821692254018921972354
load16201319197814717211880
domContentLoaded16141315196814617161872
domInteractive15115491443
firstPaint1756336863216304
backgroundConnect289284372768
firstReactRender18955401125335371
getState1143061423
initialActions317135
loadScripts16101313194414517141868
setupStore176185182038
FirefoxBrowserifyHomeuiStartup13541188170010714011558
load12071020153211212711404
domContentLoaded12071020153211212711404
domInteractive1064021034123183
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect221489132163
firstReactRender22195042327
getState74182810
initialActions001001
loadScripts11871006151711012521392
setupStore5413167
WebpackHomeuiStartup15061335205312315731752
load12911147176810513301504
domContentLoaded12911147176810513301504
domInteractive82391562394137
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21157372330
firstReactRender34296753640
getState94427928
initialActions102111
loadScripts12711130175010413091476
setupStore85304819
Benchmark value 371 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Sum of mean exceeds: 0ms | Sum of p95 exceeds: 1ms
Sum of all benchmark exceeds: 1ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 142 Bytes (0%)
  • ui: -417 Bytes (-0.01%)
  • common: 0 Bytes (0%)

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 23, 2025
@gantunesr gantunesr added this pull request to the merge queue Apr 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 23, 2025
@gantunesr gantunesr added this pull request to the merge queue Apr 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Apr 23, 2025
@gantunesr gantunesr dismissed stale reviews from ccharly and owencraston via 1647a88 April 24, 2025 00:41
@metamaskbot
Copy link
Collaborator

Builds ready [1647a88]
UI Startup Metrics (1207 ± 58 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1207106614065812461316
load104494512065310751150
domContentLoaded103994111975310701144
domInteractive17136261728
firstPaint763163120539410591116
backgroundConnect7414279
firstReactRender21164852133
getState1253171726
initialActions003001
loadScripts80269994252833908
setupStore84162814
WebpackHomeuiStartup21091710252616722192351
load16301318194113017321824
domContentLoaded16231314191812917271814
domInteractive151159101342
firstPaint1766432762206299
backgroundConnect24967122756
firstReactRender21054408121334358
getState144300301330
initialActions316134
loadScripts16191311189512817251796
setupStore216301391932
FirefoxBrowserifyHomeuiStartup1361121016179314101551
load12131036149510012751401
domContentLoaded12121036149510012741401
domInteractive1024022630116156
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2313178202248
firstReactRender24195772451
getState84345812
initialActions001001
loadScripts1191101814759912551376
setupStore7437568
WebpackHomeuiStartup14971326181110815591716
load1283114015389813541486
domContentLoaded1283113915379813531485
domInteractive81371482089128
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21155672141
firstReactRender35295553745
getState84294918
initialActions002111
loadScripts1265112515209813371464
setupStore95517829
Benchmark value 29 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 0ms | Sum of p95 exceeds: 1ms
Sum of all benchmark exceeds: 1ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 142 Bytes (0%)
  • ui: -417 Bytes (-0.01%)
  • common: 0 Bytes (0%)

@metamaskbot
Copy link
Collaborator

Builds ready [07e847c]
UI Startup Metrics (1177 ± 57 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1177106714935712071262
load101990013505610511094
domContentLoaded101389413465610461085
domInteractive17135151628
firstPaint64882135140810251083
backgroundConnect64172710
firstReactRender21155972137
getState1355791729
initialActions001001
loadScripts778673111155808844
setupStore74193812
WebpackHomeuiStartup20521671257718121972274
load15811313194813116831751
domContentLoaded15751310192113016771743
domInteractive161157111249
firstPaint1757040562208310
backgroundConnect23996142655
firstReactRender19155358119339354
getState114166161123
initialActions317134
loadScripts15701308189812916741740
setupStore1674061929
FirefoxBrowserifyHomeuiStartup13461152180812214041593
load12021030168012512651432
domContentLoaded12021030168012512651432
domInteractive1064222833122173
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21136392238
firstReactRender22195462228
getState9418518813
initialActions001001
loadScripts11831016166212512491418
setupStore6450669
WebpackHomeuiStartup14931304184911415771671
load12821137161510113591449
domContentLoaded12821137161410113591449
domInteractive85353753596125
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20144152234
firstReactRender34294533640
getState84414811
initialActions102111
loadScripts12631122159510013361433
setupStore85344812

@gantunesr gantunesr enabled auto-merge April 24, 2025 23:27
@gantunesr gantunesr added this pull request to the merge queue Apr 24, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [e2f110b]
UI Startup Metrics (1173 ± 63 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1173106213826312071313
load101490511785610411130
domContentLoaded100889811735510371124
domInteractive17124951727
firstPaint67470118040210261117
backgroundConnect6425378
firstReactRender21154972338
getState1253071727
initialActions001000
loadScripts77565791753804894
setupStore9412712813
WebpackHomeuiStartup20481678254716521362301
load15951313198812016921780
domContentLoaded15881310196412016881774
domInteractive15105191341
firstPaint1918339560245295
backgroundConnect22975132458
firstReactRender21155370129345358
getState933951118
initialActions316134
loadScripts15841308194012016861771
setupStore176290281730
FirefoxBrowserifyHomeuiStartup14001141184215614941719
load12411017165114813351547
domContentLoaded12411017165114813351546
domInteractive1053824034122175
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2413129182555
firstReactRender24195772437
getState84375816
initialActions001001
loadScripts12191002158914713071519
setupStore10419926713
WebpackHomeuiStartup1505136517759815871689
load1292115315739313601466
domContentLoaded1292115315739313601466
domInteractive82372732788130
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2215104112238
firstReactRender35295953645
getState84315924
initialActions001011
loadScripts1271113715569213351447
setupStore85435810
cc: @HowardBraham
Benchmark value 1242 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 10 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1719 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 1547 exceeds gate value 1495 for firefox browserify home p95 load
Benchmark value 1546 exceeds gate value 1495 for firefox browserify home p95 domContentLoaded
Benchmark value 1519 exceeds gate value 1475 for firefox browserify home p95 loadScripts
Sum of mean exceeds: 4ms | Sum of p95 exceeds: 206ms
Sum of all benchmark exceeds: 210ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 142 Bytes (0%)
  • ui: -417 Bytes (-0.01%)
  • common: 0 Bytes (0%)

Merged via the queue into main with commit fcb7c3b Apr 25, 2025
169 checks passed
@gantunesr gantunesr deleted the gar/refactor/remove-network branch April 25, 2025 00:11
@github-actions github-actions bot locked and limited conversation to collaborators Apr 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed team-accounts-framework Accounts Framework team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants