-
-
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
chore: Update base controller v5, logging controller v3, permission controll… #9640
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 dependencies detected. Learn more about Socket for GitHub ↗︎
|
👍 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: 3b93d7b Note
|
Bitrise✅✅✅ Commit hash: 456816b Note
|
Bitrise✅✅✅ Commit hash: 8a8da92 Note
|
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.
<3 LGTM
Quality Gate passedIssues Measures |
Description
Update:
Addressed ts errors on engine file.
Added ts expect errors for the controllers that need to be updated due to base controller version mismatch.
Fix ts issue on rpc method middleware.
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Permissions testing:
https://github.com/MetaMask/metamask-mobile/assets/46944231/be1636be-a35c-427e-9dc2-c2bdcbcd445c
Networks testing:
https://github.com/MetaMask/metamask-mobile/assets/46944231/2412e8b4-ea01-4d23-96a7-9262176ba8cf
Phishing website testing:
https://github.com/MetaMask/metamask-mobile/assets/46944231/a70c07e0-5852-49ea-9676-87b1a57b8613
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist