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.

Codecov Report

Attention: Patch coverage is 46.96970% with 35 lines in your changes are missing coverage. Please review.

Project coverage is 65.76%. Comparing base (5f22357) to head (283fd03).

@@             Coverage Diff             @@
##           develop   #24281      +/-   ##
===========================================
- Coverage    65.79%   65.76%   -0.02%     
===========================================
  Files         1364     1366       +2     
  Lines        54315    54385      +70     
  Branches     14126    14139      +13     
===========================================
+ Hits         35732    35766      +34     
- Misses       18583    18619      +36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

I haven't been able to get this modal to appear yet. Maybe my process is wrong, and not sufficient to simulate the upgrade.

I imported a fresh build from develop and disabled token autodetect. Then I changed to a build of this branch. But it doesn't show the modal because this is undefined instead of null:

I haven't been able to get this modal to appear yet. Maybe my process is wrong, and not sufficient to simulate the upgrade.

I imported a fresh build from develop and disabled token autodetect. Then I changed to a build of this branch. But it doesn't show the modal because this is undefined instead of null:

Hey @bergeron ,

It seems there's an issue with the process you're following. The modal in question is designed to show up exclusively during the onboarding/upgrade process, and it will only do so after a fresh installation of the app.

if you're doing the fresh app using develop branch, you'll not see the modal.

To replicate the issue, there are two methods. The first method applies to users who have the auto-detection feature turned off by default:

• In the preferences.js file, set the state to false (this simulates users who have token auto-detection disabled by default).
• Build and import your app using the auto-detect-token-modal branch.
• Import your account.
• You should see the modal appear right after the transaction simulation (refer to the linked video for clarity).

The second method is for users who had auto-detection enabled by default upon installing the app:

• Install your app from the build using the auto-detect-token-modal branch.
• Navigate to settings and turn off token auto-detection.
• Return to the wallet view, and the modal should appear.
• For this scenario, please refer to the provided video for more details.

Let me know if you need any further information or clarification.

I ve tested the upgrade scenario and the fresh import, the modal was displayed correctly, thnx 🙏