-
Notifications
You must be signed in to change notification settings - Fork 160
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
Verification UI / UX iteration #3829
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Quality Gate passedIssues Measures |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3829 +/- ##
========================================
Coverage 82.98% 82.99%
========================================
Files 1782 1782
Lines 44860 44881 +21
Branches 5265 5277 +12
========================================
+ Hits 37228 37247 +19
Misses 5793 5793
- Partials 1839 1841 +2 ☔ View full report in Codecov by Sentry. |
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.
LGTM, thanks!
It feels kind of weird having a 'done' action on a verification failed screen, but if it's what was designed then 🤷 .
Content
Iterate on a few screens of the verification flows
Motivation and context
Closes #3824
Screenshots / GIFs
See the recorded screenshots
Tests
Tested devices
Checklist