Skip to content
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

chore: remove unused usedNetworks state property from AppStateController #28813

Merged

Conversation

cryptodev-2s
Copy link
Contributor

@cryptodev-2s cryptodev-2s commented Nov 29, 2024

Description

Building on the work done to remove the network modal in PR, this PR finalizes the process by completely removing the unused usedNetworks state property from AppStateController.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

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.

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.

@cryptodev-2s cryptodev-2s changed the title chore: remove unused usedNetworks AppStateController property chore: remove unused usedNetworks AppStateController state property Nov 29, 2024
@cryptodev-2s cryptodev-2s force-pushed the cryptodev2s/remove-unused-userNetworks-app-state-property branch from 913d6e4 to 3c8bee1 Compare November 29, 2024 14:10
@cryptodev-2s cryptodev-2s changed the title chore: remove unused usedNetworks AppStateController state property chore: remove unused usedNetworks AppStateController state property Nov 29, 2024
@cryptodev-2s cryptodev-2s changed the title chore: remove unused usedNetworks AppStateController state property chore: remove unused usedNetworks state property from AppStateController Nov 29, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [3c8bee1]
Page Load Metrics (2164 ± 116 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint37327541992588282
domContentLoaded183726942135239115
load184827612164241116
domInteractive288841168
backgroundConnect116629199
firstReactRender16472384
getState53133972110
initialActions01000
loadScripts138622071655216104
setupStore6161031
uiStartup206830762460278134
Bundle size diffs [🚀 Bundle size reduced!]
  • background: -391 Bytes (-0.01%)
  • ui: -40 Bytes (-0.00%)
  • common: -240 Bytes (-0.00%)

sahar-fehri
sahar-fehri previously approved these changes Nov 29, 2024
salimtb
salimtb previously approved these changes Nov 29, 2024
sahar-fehri
sahar-fehri previously approved these changes Nov 29, 2024
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@cryptodev-2s cryptodev-2s added this pull request to the merge queue Nov 29, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [593e03d]
Page Load Metrics (2008 ± 102 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint26924471704694333
domContentLoaded16642383197518890
load166925512008213102
domInteractive26139422612
backgroundConnect7161323417
firstReactRender16111282110
getState712231103718
initialActions01000
loadScripts12711798152215273
setupStore684162110
uiStartup188334032333349167
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 535 Bytes (0.01%)
  • ui: -40 Bytes (-0.00%)
  • common: -240 Bytes (-0.00%)

Merged via the queue into develop with commit 7143c96 Nov 29, 2024
75 checks passed
@cryptodev-2s cryptodev-2s deleted the cryptodev2s/remove-unused-userNetworks-app-state-property branch November 29, 2024 17:26
@github-actions github-actions bot locked and limited conversation to collaborators Nov 29, 2024
@metamaskbot metamaskbot added the release-12.10.0 Issue or pull request that will be included in release 12.10.0 label Nov 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-12.10.0 Issue or pull request that will be included in release 12.10.0 team-assets team-wallet-framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants