-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 @metamask/swaps-controller
to 12.0.0
#12378
base: main
Are you sure you want to change the base?
Conversation
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. |
fa4f400
to
f0c6e60
Compare
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@metamask/[email protected] |
14b8bf9
to
58fe119
Compare
58fe119
to
eb6f6e9
Compare
swaps-controller
to test removal of global providerswaps-controller
to test removal of global provider
Need to add missing unit tests in order to make SonarCloud pass on this, but all of the existing unit tests should pass and you should be able to follow the manual testing steps listed in the PR description. |
Quality Gate passedIssues Measures |
swaps-controller
to test removal of global providerswaps-controller
to test removal of global provider
swaps-controller
to test removal of global provider@metamask/swaps-controller
to 12.0.0
Description
This PR upgrades
@metamask/swaps-controller
to 12.0.0.The most notable change here is that the controller has been updated to mostly not rely on the global provider; instead
networkClientId
is passed to the various methods that need it.See the full changelog here: TODO
Related issues
Fixes #12470.
Also see:
Manual testing steps
Screenshots/Recordings
The Swaps flow should work exactly like it does now.
Here is a video demonstrating fetching Swaps quotes:
Screen.Recording.2024-11-22.at.12.25.33.PM.mov
Pre-merge author checklist
Pre-merge reviewer checklist