Closed infiniteflower closed 4 days ago
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.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code
All modified and coverable lines are covered by tests :white_check_mark:
Please upload report for BASE (
release/7.24.4@b1f0550
). Learn more about missing BASE report.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
LGTM for QA ✅ I tested iOS and Android, with smart transactions on and off.
https://github.com/MetaMask/metamask-mobile/assets/124314512/82407752-960d-4662-9d19-ae665bb20c9f https://github.com/MetaMask/metamask-mobile/assets/124314512/f36c47f1-aa6a-4e82-a513-1b5a3187b332
LGTM for QA, I verified the fix in RC 2
No release label on PR. Adding release label release-7.24.4 on PR, as PR was added to branch 7.24.4 when release was cut.
Description
This is the cherry pick PR for #10099
This PR fixes an issue with Ramps and STX missing an origin. This will disable STX for all Ramps Send transactions.
Related issues
https://github.com/MetaMask/metamask-mobile/issues/10100
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Description
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist