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: Bump selected-network-controller to v21 #13109

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mcmire
Copy link
Contributor

@mcmire mcmire commented Jan 21, 2025

Description

This version is functionally equivalent to v19. v20 enabled the Per-Dapp Selected Network feature by default and removed the option to disable it. This would have broken Mobile since that feature has not been integrated here yet. That has been reverted to v21, so this commit essentially ensures that we are using the latest version of selected-network-controller for parity with Extension.

Related issues

Fixes #13104.

Manual testing steps

No manual testing steps needed since the Per-Dapp Selected Network feature is still behind a feature flag.

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.

This version is functionally equivalent to v19. v20 enabled the Per-Dapp
Selected Network feature by default and removed the option to disable
it. This would have broken Mobile since that feature has not been
integrated here yet. That has been reverted to v21, so this commit
essentially ensures that Mobile is using the latest version of
`selected-network-controller`.
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.

@mcmire mcmire changed the title Bump selected-network-controller to v21 chore: Bump selected-network-controller to v21 Jan 21, 2025
Copy link

socket-security bot commented Jan 21, 2025

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@metamask/[email protected] 🔁 npm/@metamask/[email protected] None 0 137 kB metamaskbot
npm/@metamask/[email protected] 🔁 npm/@metamask/[email protected] None 0 37.1 kB metamaskbot

View full report↗︎

@mcmire mcmire added the No QA Needed Apply this label when your PR does not need any QA effort. label Jan 21, 2025
@mcmire mcmire added the No E2E Smoke Needed If the PR does not need E2E smoke test run label Jan 22, 2025
@mcmire mcmire marked this pull request as ready for review January 22, 2025 16:38
@mcmire mcmire requested review from a team as code owners January 22, 2025 16:38
@mcmire mcmire requested a review from a team January 22, 2025 16:39
@mcmire mcmire enabled auto-merge January 22, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No E2E Smoke Needed If the PR does not need E2E smoke test run No QA Needed Apply this label when your PR does not need any QA effort. team-wallet-framework
Projects
Status: Needs dev review
Development

Successfully merging this pull request may close these issues.

Update metamask/selected-network-controller to version 21.0.0
2 participants