-
Notifications
You must be signed in to change notification settings - Fork 5k
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
fix (cherry-pick): incomplete transactions on startup (#26963) #27537
fix (cherry-pick): incomplete transactions on startup (#26963) #27537
Conversation
Upgrade the `TransactionController` to `37.0.0`. Prevent various scenarios triggering the `signed or submitted` alert . Remove unapproved transaction fixtures as no longer supported.
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] |
396c68c
to
2cedf71
Compare
@metamaskbot update-policies |
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
3a51546
to
fb5b3d5
Compare
fb5b3d5
to
ba77578
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## Version-v12.4.0 #27537 +/- ##
================================================
Coverage 70.05% 70.05%
================================================
Files 1422 1422
Lines 49677 49677
Branches 13887 13887
================================================
Hits 34800 34800
Misses 14877 14877 ☔ View full report in Codecov by Sentry. |
Builds ready [ba77578]
Page Load Metrics (1665 ± 92 ms)
|
…saction-controller-startup
Builds ready [c942cc0]
Page Load Metrics (1683 ± 53 ms)
|
…saction-controller-startup
Description
This PR cherry-picks #26963
Related issues
Fixes: #27486
Manual testing steps
Screenshots/Recordings
clean-unapproved-tx.webm
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist