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

Release/148.0.0 #4260

Merged
merged 11 commits into from
May 6, 2024
Merged

Release/148.0.0 #4260

merged 11 commits into from
May 6, 2024

Conversation

adonesky1
Copy link
Contributor

@metamask/selected-network-controller

[13.0.0]

Changed

  • getProviderAndBlockTracker now returns the NetworkController's globally selected network client proxy if the domain arg is either metamask or a snap (identified as starting with npm: or local:) (#4259)
  • BREAKING: Now when setNetworkClientIdForDomain is called with a snap's domain (identified as starting with npm: or local:), the domain will not be added to state and no proxy will be created for this domain in the domainProxyMap (#4258)
    • In order to remove snaps that made it into domains state prior to this change, consumers will need to run a migration.

@adonesky1 adonesky1 requested a review from a team as a code owner May 6, 2024 18:19
@adonesky1 adonesky1 requested review from mcmire and Gudahtt May 6, 2024 18:22
adonesky1 and others added 2 commits May 6, 2024 13:26
yarn.lock Outdated Show resolved Hide resolved
@adonesky1 adonesky1 requested a review from Gudahtt May 6, 2024 18:52
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
Gudahtt
Gudahtt previously approved these changes May 6, 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!

@Gudahtt Gudahtt dismissed their stale review May 6, 2024 19:25

Version bump needed

@adonesky1 adonesky1 merged commit 02de743 into main May 6, 2024
139 checks passed
@adonesky1 adonesky1 deleted the release/148.0.0 branch May 6, 2024 19:26
@Gudahtt
Copy link
Member

Gudahtt commented May 6, 2024

Sorry thought I spotted a problem there, but I misread something. Ignore the dismissal and re-approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants