-
Notifications
You must be signed in to change notification settings - Fork 499
New submission flow: Your consent screen (EXPOSUREAPP-3735) #1767
New submission flow: Your consent screen (EXPOSUREAPP-3735) #1767
Conversation
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.
Looks good to me 👍
Obviously, we need to finalize the flow in terms of navigation once the submission PRs are merged. Then we can test how they all work together. As a single PR though it looks good to me.
Corona-Warn-App/src/main/res/layout/fragment_submission_your_consent.xml
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_submission_your_consent.xml
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_submission_your_consent.xml
Show resolved
Hide resolved
This looks good to me. There are some minor design differences to figma but they are includes that were already added in a different PR. I'm not sure if these should be changed here so once all your checks pass I'll approve it. |
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.
Re-review after lint changes. LGTM 👍
…-your-consent-screen # Conflicts: # Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/viewmodel/SubmissionFragmentModule.kt
4d3c5b4
Kudos, SonarCloud Quality Gate passed! |
Your consent screen.
Navigation would be placed in separate PR.