-
Notifications
You must be signed in to change notification settings - Fork 155
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
Recovery key formatting and wording iteration #3409
Conversation
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.
Not sure why this is recorded again, the timestamp is not rendered the same way :/.
There is no change in this PR about this.
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.
This was recorded from a fork, maybe their CI setup is somehow in a different TZ?
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Quality Gate passedIssues Measures |
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.
Thanks!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3409 +/- ##
========================================
Coverage 82.62% 82.62%
========================================
Files 1694 1694
Lines 39685 39686 +1
Branches 4831 4830 -1
========================================
+ Hits 32788 32789 +1
Misses 5205 5205
Partials 1692 1692 ☔ View full report in Codecov by Sentry. |
Content
If a passphrase is entered by the user the
RecoveryKeyVisualTransformation
does not apply. This PR is adding 2 screenshots to demo this.Also stop using the wording
screen_recovery_key_confirm_key_label
(Recovery key or passcode
), it was wrong. Just useRecovery key
(similar to iOS and compliant with design)Motivation and context
Improve wording and screenshot test.
Screenshots / GIFs
Tests
Tested devices
Checklist