-
-
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
feat: Update transaction controller to v8.0.1 #8866
Conversation
…pdate transaction controller to version 8.0.1, both patches were updated
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/approval-controller@3.4.0, npm/@metamask/transaction-controller@7.1.0, npm/zxcvbn@4.4.2 |
👍 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. |
…controller be updated
Bitrise❌❌❌ Commit hash: 3a9c12c Note
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8866 +/- ##
=======================================
Coverage 41.66% 41.66%
=======================================
Files 1271 1271
Lines 30906 30906
Branches 3088 3088
=======================================
Hits 12877 12877
Misses 17250 17250
Partials 779 779 ☔ View full report in Codecov by Sentry. |
Bitrise✅✅✅ Commit hash: c52106c Note
|
QA build, physical device Samsung s24+, Sepolia, Linea Goerli:
|
Quality Gate passedIssues Measures |
Description
This PR aims to update the transaction controller to version 8.0.1.
The Approval Controller was also updated on this PR to the version of 3.5.0 and approval controller patched was updated.
All the changes are present in this core branch for easy review: patch/mobile-transaction-controller-8-0-1
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
transactions: https://recordit.co/mpcRkPQElC
sign: http://recordit.co/yPI78aEozu
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist