-
Notifications
You must be signed in to change notification settings - Fork 984
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
Show proper error when bio metric fails #20233
Conversation
Jenkins BuildsClick to see older builds (8)
|
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.
Do we need to handle the error for iOS as well?
hi @clauxx, Thank you very much for approving PR. I currently don't have access to ios device with touch-id and don't know what will be error message for ios for too many attempts. We can add for ios later. |
76% of end-end tests have passed
Not executed tests (1)Failed tests (8)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestWalletMultipleDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (4)Click to expandClass TestWalletOneDevice:
Class TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (39)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestWalletOneDevice:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
|
@status-im/mobile-qa Please can you check failed e2e tests. |
ba7b967
to
e2c865d
Compare
e2c865d
to
cc829c6
Compare
12% of end-end tests have passed
Failed tests (7)Click to expandClass TestWalletMultipleDevice:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityMultipleDeviceMerged:
Passed tests (1)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
|
@Parveshdhull thank you for the PR. Failed e2e are not PR related. Ready for merge. |
fixes #20007
leftover from #19989 (comment)
Testing
PR is manually tested and safe to merge. I only added request-manual-qa to make sure error message gets double checked.
Too many attempts. Enter the password manually or try again after some time
UPD: Adding skip-manual-qa and merging. We can always update msg if it needs to.
status: ready