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

Conversation

@chris-cwa
Copy link
Contributor

No description provided.

@chris-cwa chris-cwa added maintainers Tag pull requests created by maintainers sprint5 labels Sep 9, 2020
@chris-cwa chris-cwa requested review from a team and harambasicluka September 9, 2020 10:36
@chris-cwa chris-cwa changed the title Fixed Message for bad TAN - EXPOSUREAPP-2552 Fixed Message for bad TAN - (EXPOSUREAPP-2552) Sep 9, 2020
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,
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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)...

Copy link
Contributor

@harambasicluka harambasicluka left a 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>
Copy link
Contributor

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 -->

@chris-cwa chris-cwa marked this pull request as draft September 11, 2020 09:20
@harambasicluka
Copy link
Contributor

Updated strings and Dialog title:
image

@harambasicluka harambasicluka changed the title Fixed Message for bad TAN - (EXPOSUREAPP-2552) Fixed Message for bad TAN (EXPOSUREAPP-2552) Sep 15, 2020
@harambasicluka
Copy link
Contributor

new string:
image

Copy link
Contributor

@janetback janetback left a comment

Choose a reason for hiding this comment

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

UA approved

Copy link
Contributor

@BMItr BMItr left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@harambasicluka harambasicluka merged commit d06b043 into dev Sep 15, 2020
@harambasicluka harambasicluka deleted the EXPOSUREAPP_2552 branch September 15, 2020 13:44
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 sprint5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants