-
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
[Fix] QR scanner scanning QR when user moved to next screen #16012
Conversation
Jenkins BuildsClick to see older builds (12)
|
This is due to the navigational structure of the stack navigation. Let's say we have screens A, B, C,...etc. On stack navigation, when the user moves from That's why the camera on the previous screen is active. Once the syncing is completed successfully, we reset the whole stack with When the QR scans any sync code, it moves to To prevent the QR scanner to process any further data. I believe checking the |
e248993
to
fa5d3a2
Compare
82% of end-end tests have passed
Failed tests (6)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
Passed tests (27)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
|
fa5d3a2
to
1197087
Compare
100% of end-end tests have passed
Passed tests (2)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
|
Hi @smohamedjavid . Thank you for PR. ISSUE 1: Continuous QR code scanning even after "Oops something wrong" page is displayedSteps to reproduce:
Actual result:The IOS device continues to scan the QR code. (I don't tap 'try again' button on the record) senc_problem.mp4Expected result:The IOS device should stop scanning the QR code until the "Try again" button is tapped. Device:
|
1197087
to
fc5786e
Compare
@VladimrLitvinenko - Kindly retest. |
67% of end-end tests have passed
Failed tests (11)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Passed tests (22)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
85% of end-end tests have passed
Failed tests (5)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (28)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
hi @smohamedjavid Thank you for the fixes. PR is ready to be merged |
Signed-off-by: Mohamed Javid <19339952+smohamedjavid@users.noreply.github.com>
Signed-off-by: Mohamed Javid <19339952+smohamedjavid@users.noreply.github.com>
ec7a57a
to
8af6402
Compare
fixes #15970
Summary
This PR fixes the QR scanner that scans the QR code when the user is moved to the next screen (
:syncing-progress
).The bug was caused when due to the navigational stack. The "QR scanner" screen will be behind the "Syncing devices" screen, and the QR scanner reset after 3 seconds to scan/process any QR code. The fix was to ensure the scanner scan/process the data only when the user is on the "QR scanner" screen.
Platforms
Steps to test
Sign in
Something went wrong
screen more than once.status: ready