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

UQS update submission flow screen order (EXPOSUREAPP-9332) #4061

Merged
merged 10 commits into from
Sep 15, 2021

Conversation

SamuraiKek
Copy link
Contributor

Order of the screens in the PCR/RAT submission flow is updated to match flow of UQS.

The PR addresses the second dot of the acceptance criteria from Jira but there's no thorough description provided for every scenario. I did my best to keep a logical flow but please test thorougly because I might have missed something.

Testing flow:

  • Register a new positive test by scanning QR code.

  • First screen that shows up after pressing "Scan QR Code" is the QR scan screen.

  • After scanning the QR, you are taken to the "Your Consent" screen.

    • opening the data privacy page works.
    • pressing the x button takes you back to QR scanner.
    • pressing the OS back button takes you back to the QR scanner.
    • accepting the consent takes you to the "Covid Test Certificate" screen where you can request a test cert.
  • On the "Covid Test Certificate" screen, after selecting either option you get to "Your result is Available" screen.

  • Pressing next takes you to the result screen.

  • Register another test of same type.

  • Flow is the same as the one above but after the consent screen, you first go to the deletion screen to delete the already registered test.

# Conflicts:
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/qrcode/scan/SubmissionQRCodeScanFragment.kt
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/qrcode/scan/SubmissionQRCodeScanViewModel.kt
@SamuraiKek SamuraiKek added maintainers Tag pull requests created by maintainers author merge PR to be merged by author labels Sep 14, 2021
@SamuraiKek SamuraiKek added this to the 2.11.0 milestone Sep 14, 2021
@SamuraiKek SamuraiKek requested a review from a team September 14, 2021 09:35
@AlexanderAlferov AlexanderAlferov self-assigned this Sep 14, 2021
@mtwalli
Copy link
Contributor

mtwalli commented Sep 14, 2021

We need to talk ?

@mtwalli mtwalli self-assigned this Sep 14, 2021
@corona-warn-app corona-warn-app deleted a comment from SamuraiKek Sep 14, 2021
@corona-warn-app corona-warn-app deleted a comment from SamuraiKek Sep 14, 2021
@sonarcloud
Copy link

sonarcloud bot commented Sep 15, 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

9.4% 9.4% Coverage
0.0% 0.0% Duplication

@mtwalli mtwalli merged commit 00f73df into release/2.11.x Sep 15, 2021
@mtwalli mtwalli deleted the feature/9332-uqs-update-info-screens branch September 15, 2021 10:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
author merge PR to be merged by author maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants