-
Notifications
You must be signed in to change notification settings - Fork 97
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
feat: [IOPID-1507,IOPID-1533] Integrate new DS in Pin Screen flow #5737
Conversation
…rding-app-pin-screen
Affected stories
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5737 +/- ##
==========================================
+ Coverage 48.42% 49.62% +1.19%
==========================================
Files 1488 1653 +165
Lines 31617 32916 +1299
Branches 7669 8009 +340
==========================================
+ Hits 15311 16333 +1022
- Misses 16238 16521 +283
+ Partials 68 62 -6
... and 613 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
@mariateresaventura copy review needed 🙏🏼 |
ts/components/screens/PinCreation/usePinValidationBottomSheet.tsx
Outdated
Show resolved
Hide resolved
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.
ts/components/screens/PinCreation/usePinValidationBottomSheet.tsx
Outdated
Show resolved
Hide resolved
ts/components/screens/PinCreation/usePinValidationBottomSheet.tsx
Outdated
Show resolved
Hide resolved
ts/components/screens/PinCreation/usePinValidationBottomSheet.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Fabio Bombardi <16268789+shadowsheep1@users.noreply.github.com>
Co-authored-by: Fabio Bombardi <16268789+shadowsheep1@users.noreply.github.com>
Co-authored-by: Fabio Bombardi <16268789+shadowsheep1@users.noreply.github.com>
Co-authored-by: Fabio Bombardi <16268789+shadowsheep1@users.noreply.github.com>
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 🚀
Short description
Integrated new DS in
PinScreen
componentList of changes proposed in this pull request
Profile
section with a non-blocking toast message**Note
** The need to restart the application after the pin was changed comes from this PR to make it immediately available in app. Today this behavior is not necessary anymore because the pin is now retrieved directly from
react-native-keychain
when needed.Test
E2E
Tip
Screens
iOS
iOS-onboarding-pin-screen.mov
iOS-profile-pin-screen.mov
iOS-pin-screen-validation-error.mov
iOS-pin-screen-confirmation-error.mov
Android
android-onboarding-pin-screen.mp4
android-profile-pin-screen.mp4
android-pin-screen-validation-error.mp4
android-pin-screen-confirmation-error.mp4
Accessibility iOS & Android
iOS.a11y.-onboardin-pin-screen.mov
android-a11y-onboarding-pin-screen.mp4
iOS.a11y.-profile-pin-screen.mov
android-a11y-profile-pin-screen.mp4
iOS.a11y.-pin-screen-validation-error.mov
android-a11y-pin-screen-validation-error.mp4
iOS.a11y.-pin-screen-confirmation-error.mov
android-a11y-pin-screen-confirmation-error.mp4
How to test
Since this component is used in two sections of the app -
Onboarding
andProfile
flows - there are two different ways in which it can be tested.Profile section
This is the easiest way in which it can be tested:
Profile > Security > Change the unlock code
Onboarding section
Testing in local environment - clone io-dev-api-server repo if you haven't did it yet and follow the instructions - go in
config/config.json
and inprofile.attrs
add a fiscal code underfiscal_code
key (must be 16 characters long), restart you backend and then restart the application to login again and display the onboarding flow with the new Pin Screen.Tip
To try multiple times repeat the process changing at least one
fiscal_code
character