-
-
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
fix: Fix the QR code accounts disappear when password change. #8741
Conversation
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 ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8741 +/- ##
==========================================
- Coverage 41.61% 41.61% -0.01%
==========================================
Files 1269 1269
Lines 30869 30869
Branches 3086 3086
==========================================
- Hits 12847 12846 -1
- Misses 17245 17246 +1
Partials 777 777 ☔ View full report in Codecov by Sentry. |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/fea43d1c-c2f8-4e3a-87c4-3aab97669a26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me on the code side.
This PR needs a QA pass.
Tested locally and working as expected:- |
Bitrise✅✅✅ Commit hash: 8737a77 Note
|
I have read the CLA Document and I hereby sign the CLA |
Britrise e2e pass even the github action pipeline show failed. MM team will fix the github action e2e tests issue in |
Quality Gate passedIssues Measures |
Description
This PR will fix #8694 and #8693, the fix contain following change.
After some
Related issues
Fixes: #8694
#8693
Manual testing steps
For #8694
For #8693
Screenshots/Recordings
Before
PasswordChangeQRDisappear.mov
After
Screen_Recording_20240227_173031_MetaMask.mp4
Pre-merge author checklist
Pre-merge reviewer checklist