Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Update external qr scan flow (EXPOSUREAPP-9463) #4086

Merged
merged 8 commits into from
Sep 20, 2021

Conversation

SamuraiKek
Copy link
Contributor

Scanning a RAT QR with an external scanner will bring up the deletion screen in case there's an already registered RAT test inside the app or the consent screen in case there isn't.

Test the flow with & without any RATs saved in the app.

@SamuraiKek SamuraiKek added the maintainers Tag pull requests created by maintainers label Sep 17, 2021
@SamuraiKek SamuraiKek added this to the 2.11.0 milestone Sep 17, 2021
@SamuraiKek SamuraiKek requested a review from a team September 17, 2021 13:53
@mtwalli mtwalli self-assigned this Sep 17, 2021
@AlexanderAlferov AlexanderAlferov self-assigned this Sep 20, 2021
@sonarcloud
Copy link

sonarcloud bot commented Sep 20, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

3.6% 3.6% Coverage
15.0% 15.0% Duplication

Copy link
Contributor

@AlexanderAlferov AlexanderAlferov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Copy link
Contributor

@mtwalli mtwalli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mtwalli mtwalli merged commit 45ad1c1 into release/2.11.x Sep 20, 2021
@mtwalli mtwalli deleted the feature/9463-update-external-qr-scan-flow branch September 20, 2021 13:44
@mtwalli mtwalli restored the feature/9463-update-external-qr-scan-flow branch September 20, 2021 13:44
@mtwalli mtwalli deleted the feature/9463-update-external-qr-scan-flow branch September 20, 2021 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants