-
Notifications
You must be signed in to change notification settings - Fork 499
Add warning deletion screen for Antigen and PCR (EXPOSUREAPP-6032) #2787
Add warning deletion screen for Antigen and PCR (EXPOSUREAPP-6032) #2787
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. Spacing and layout looks as in the design.
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.
Just small changes - everything else looks nice 🚀
Corona-Warn-App/src/main/res/layout/fragment_submission_deletion_warning.xml
Outdated
Show resolved
Hide resolved
...java/de/rki/coronawarnapp/ui/submission/deletionwarning/SubmissionDeletionWarningFragment.kt
Outdated
Show resolved
Hide resolved
<!-- YTXT: Body for rapid test submission deletion warning --> | ||
<string name="submission_deletion_warning_body_antigen_test">"Sie haben bereits einen Schnelltest erfasst. Die App kann maximal einen Schnelltest und einen PCR-Test gleichzeitig verwalten. Wenn Sie einen weiteren Schnelltest erfassen, wird der erste Schnelltest aus der App gelöscht."</string> | ||
<!-- YTXT: Headline for PCR test submission deletion warning --> | ||
<string name="submission_deletion_warning_headline_pcr_test">"Sie haben bereits einen PCR-Test registriert."</string> |
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.
See other comment
Hi @axelherbstreith, @SabineLoss @vaubaehn, although @vaubaehn 's distinction is correct, this is a very technical distinction which I believe does not need to be made on the UI itself. @SabineLoss , you are right, we should stay consistent here. In most instances we have used 'registered', which is why I suggest changing to 'registrieren' in this case too. |
322c079
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.
UA approved
Kudos, SonarCloud Quality Gate passed! |
Currently only added the new screen. Connection to the submission flow will be done in a separate PR.