metamask-extension icon indicating copy to clipboard operation
metamask-extension copied to clipboard

fix: Allow users to remove linea from networks list

Open gambinish opened this issue 1 year ago • 13 comments

Description

Users should be allowed to remove Linea from networks list. When Linea is removed from default network list, it should appear within the Popular Networks section (be suggested to them). So, we need to also add it to FEATURED_RPCS

Open in GitHub Codespaces

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MMASSETS-387

Manual testing steps

  1. Go to networks list, ensure that you can remove linea, even though it is a default network.

Screenshots/Recordings

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.

gambinish avatar Sep 30 '24 21:09 gambinish

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.

github-actions[bot] avatar Sep 30 '24 21:09 github-actions[bot]

Builds ready [8c9bf20]
Page Load Metrics (1848 ± 127 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint28022661659517248
domContentLoaded145822561813260125
load146622661848264127
domInteractive248241199
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 0 Bytes (0.00%)

metamaskbot avatar Sep 30 '24 21:09 metamaskbot

Let's make sure if linea is deleted, it appears as a popular network so people have the ability to re-add it. The list of popular networks should be defined by FEATURED_RPCS so you can start with trying to add it there and see what breaks.

bergeron avatar Oct 02 '24 16:10 bergeron

Builds ready [5755063]
Page Load Metrics (1937 ± 138 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint157726351927288138
domContentLoaded156124911885251120
load157026411937288138
domInteractive197542189
backgroundConnect9208475024
firstReactRender723891206431
getState5116233014
initialActions01000
loadScripts11391803140216077
setupStore12110292612
uiStartup174231422198405195

metamaskbot avatar Oct 15 '24 16:10 metamaskbot

Let's make sure if linea is deleted, it appears as a popular network so people have the ability to re-add it. The list of popular networks should be defined by FEATURED_RPCS so you can start with trying to add it there and see what breaks.

https://github.com/user-attachments/assets/742db780-91e0-46ab-9f8d-78a21f22c766

As far as I can tell, this is working as expected since Linea is already added as an entry of FEATURED_RPCS

Am I missing something here?

gambinish avatar Oct 15 '24 19:10 gambinish

Builds ready [3e25304]
Page Load Metrics (1919 ± 127 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint157326881917253121
domContentLoaded156326791875236113
load157126901919265127
domInteractive18121472412
backgroundConnect8213445728
firstReactRender472141215727
getState4110233417
initialActions01000
loadScripts114021901405215103
setupStore10128533718
uiStartup181532082257403193
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 0 Bytes (0.00%)

metamaskbot avatar Oct 15 '24 19:10 metamaskbot

Builds ready [af20158]
Page Load Metrics (1884 ± 103 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint16932393189320398
domContentLoaded16622291184819594
load167023941884214103
domInteractive17246635828
backgroundConnect11159393617
firstReactRender46198923818
getState587252914
initialActions01000
loadScripts12221837139717684
setupStore1188272512
uiStartup186830632111285137
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 0 Bytes (0.00%)

metamaskbot avatar Oct 16 '24 18:10 metamaskbot

My bad. I had the changes to FEATURED_RPCS on my local 😑

Should be updated now.

gambinish avatar Oct 16 '24 19:10 gambinish

Builds ready [92e1d89]
Page Load Metrics (2371 ± 418 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint438560922961033496
domContentLoaded167649642323833400
load169452222371871418
domInteractive17154724220
backgroundConnect13323536732
firstReactRender4956314611756
getState6312376632
initialActions01000
loadScripts123338311765688331
setupStore12105433115
uiStartup1862653927411125540
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 211 Bytes (0.00%)

metamaskbot avatar Oct 16 '24 20:10 metamaskbot

Builds ready [c21904c]
Page Load Metrics (1978 ± 164 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint171527931975327157
domContentLoaded165725801931289139
load171329301978342164
domInteractive198450189
backgroundConnect9379478039
firstReactRender472151085325
getState5112293316
initialActions01000
loadScripts123319731466221106
setupStore10198434522
uiStartup189342392291588283
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 251 Bytes (0.00%)

metamaskbot avatar Oct 16 '24 22:10 metamaskbot

Builds ready [cc8db41]
Page Load Metrics (1947 ± 147 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint171027611923278133
domContentLoaded167625771869224107
load171428591947306147
domInteractive278546178
backgroundConnect114257510048
firstReactRender473371248340
getState5192294522
initialActions01000
loadScripts12411879138415675
setupStore11221525727
uiStartup188542362296607292
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 251 Bytes (0.00%)

metamaskbot avatar Oct 17 '24 01:10 metamaskbot

Builds ready [715d648]
Page Load Metrics (1959 ± 154 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint165026751942256123
domContentLoaded16262207187417383
load163630561959322154
domInteractive15229745928
backgroundConnect87928017383
firstReactRender462181104622
getState4134223517
initialActions00000
loadScripts11931688140414972
setupStore11109413216
uiStartup183732992239433208
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 251 Bytes (0.00%)

metamaskbot avatar Oct 17 '24 21:10 metamaskbot

Builds ready [b356c50]
Page Load Metrics (1976 ± 172 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint60030331839535257
domContentLoaded162230061943337162
load163230361976357172
domInteractive26119532512
backgroundConnect7242375125
firstReactRender45117892311
getState495192512
initialActions01000
loadScripts116920151441249120
setupStore1162272010
uiStartup179432552190398191
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 251 Bytes (0.00%)

metamaskbot avatar Oct 21 '24 21:10 metamaskbot

Builds ready [b356c50]
Page Load Metrics (1976 ± 172 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint60030331839535257
domContentLoaded162230061943337162
load163230361976357172
domInteractive26119532512
backgroundConnect7242375125
firstReactRender45117892311
getState495192512
initialActions01000
loadScripts116920151441249120
setupStore1162272010
uiStartup179432552190398191
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 251 Bytes (0.00%)

metamaskbot avatar Oct 21 '24 21:10 metamaskbot

Builds ready [bf5f6d2]
Page Load Metrics (2279 ± 89 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint29625712078603289
domContentLoaded19432509224217182
load19492559227918689
domInteractive3193552110
backgroundConnect9123383115
firstReactRender732911295526
getState674312311
initialActions01000
loadScripts14001845165414067
setupStore1296372311
uiStartup223831632587239115
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0.00%)
  • ui: -39 Bytes (-0.00%)
  • common: 251 Bytes (0.00%)

metamaskbot avatar Oct 22 '24 17:10 metamaskbot