MetaMask / metamask-mobile

Mobile web browser providing access to websites that use the Ethereum blockchain
https://metamask.io
Other
2.04k stars 1.06k forks source link

fix: Always mark the STX Opt In modal as seen #10064

Closed dan437 closed 1 week ago

dan437 commented 1 week ago

Description

It seems that after a redirection to Settings was added when a user clicks on "Manage in settings", it wasn't marking the modal as seen. This PR fixes it.

Related issues

Fixes:

Manual testing steps

  1. Set up the app from scratch
  2. Select "Enable" or "Manage in Settings" on the STX Opt In modal
  3. You will not see the modal again, even when you switch to a different network and then back to Ethereum Mainnet

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

github-actions[bot] commented 1 week 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.

sleepytanya commented 1 week ago

Bitrise build https://app.bitrise.io/build/1577a516-9b02-448b-86ba-64a697e0e49d?tab=artifacts Samsung S24+ STX opt-in modal displayed only once:

https://github.com/MetaMask/metamask-mobile/assets/104780023/4c34cea5-0c0d-4eb6-8453-9ac39caa1e56

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

bkirb commented 1 week ago

https://github.com/MetaMask/metamask-mobile/assets/124314512/d0f90ce5-7798-46be-b2ba-ec4d6850173e

LGTM for QA, here is a screen recording