-
-
Notifications
You must be signed in to change notification settings - Fork 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
feat: Upgrade signature Controller to v14 #9651
Conversation
…er v9, phishing controller v9, addressed ts errors on engine file and added ts expect errors for the controllers that need to be updated due to base controller version mismatch, also fix ts issue on rpcmethod middle ware
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. |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@metamask/[email protected] |
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
Bitrise✅✅✅ Commit hash: 13e80c2 Note
|
Bitrise✅✅✅ Commit hash: 4455e43 Note
|
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Awesome job!
If you can just add the manual steps and a recording of the behaviour of signing
Description
Update Signature Controller to v^14
Changelog: https://github.com/MetaMask/core/blob/%40metamask/signature-controller%4016.0.0/packages/signature-controller/CHANGELOG.md
These two peer dependencies:
These two controllers are being used inside Signature Controller to import types.
These types have not been changed during the latest upgrades to these controllers, thus not being a breaking changes for Signature Controller V14.
Related issues
Manual testing steps
Screenshots/Recordings
Before
After
Recording.11.mp4
Pre-merge author checklist
Pre-merge reviewer checklist