-
Notifications
You must be signed in to change notification settings - Fork 487
Fixed Message for bad TAN (EXPOSUREAPP-2552) #1131
Conversation
| requireActivity(), | ||
| R.string.submission_error_dialog_web_test_paired_title, | ||
| R.string.submission_error_dialog_web_test_paired_body, | ||
| R.string.submission_error_dialog_web_test_paired_body_tan, |
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.
there are already related Strings..
If you want, you can use them:
is BadRequestException -> DialogHelper.DialogInstance(
requireActivity(),
R.string.submission_error_dialog_web_tan_invalid_title,
R.string.submission_error_dialog_web_tan_invalid_body,
R.string.submission_error_dialog_web_tan_invalid_button_positive,
null,
true,
::goBack
)
Kind Regards.
Bernd
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.
Hm... but this key constant has not the value as in https://jira.itc.sap.com/browse/EXPOSUREAPP-2552.
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.
submission_error_dialog_web_tan_invalid_body = Die erzeugte Übermittlungs-TAN ist ungültig. Bitte kontaktieren Sie die technische Hotline über App-Informationen → Technische Hotline.
but in the screenshot in the Ticket it says otherwise.
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.
Luka just updated the ticket (it was confusing)...
harambasicluka
left a comment
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.
lgtm, just two small changes.
| <string name="submission_error_dialog_web_test_paired_title">"QR-Code ist ungültig"</string> | ||
| <!-- XMSG: Dialog body for already paired test error --> | ||
| <string name="submission_error_dialog_web_test_paired_body">"Der QR-Code ist ungültig oder wurde bereits auf einem Smartphone registriert. Ihr Testergebnis bekommen Sie vom Testcenter oder Labor, unabhängig von der Gültigkeit des QR-Codes. Wenn Ihr Test positiv ausfällt, bekommen Sie vom Gesundheitsamt eine Mitteilung."</string> | ||
| <string name="submission_error_dialog_web_test_paired_body_tan">"Der QR-Code / Die TAN ist ungültig oder wurde schon verwendet. Bitte versuchen Sie es erneut oder kontaktieren Sie die technische Hotline über App-Informationen → Technische Hotline."</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.
please add a translation comment, e.g.
<!-- XMSG: Dialog body for already paired test via tan - error -->
janetback
left a comment
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
BMItr
left a comment
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.
LGTM
|
Kudos, SonarCloud Quality Gate passed!
|
No description provided.