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

Minor string fixes (DEV) #2629

Merged
merged 2 commits into from
Mar 16, 2021
Merged

Conversation

janetback
Copy link
Contributor

@janetback janetback commented Mar 16, 2021

Description

Changed 'Nachname, Vorname' to just 'Name' (specific request from RKI);
Changed punctuation on risk card;

@janetback janetback requested review from ralfgehrer and a team March 16, 2021 14:15
d4rken
d4rken previously approved these changes Mar 16, 2021
@d4rken d4rken self-assigned this Mar 16, 2021
@d4rken d4rken added this to the 1.15.0 milestone Mar 16, 2021
@d4rken d4rken added the maintainers Tag pull requests created by maintainers label Mar 16, 2021
@janetback janetback marked this pull request as draft March 16, 2021 14:20
@janetback
Copy link
Contributor Author

Need to add some other minor corrections (changes to punctuation) -> setting to draft.

@sonarcloud
Copy link

sonarcloud bot commented Mar 16, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@@ -12,7 +12,7 @@
<string name="contact_diary_person_list_no_items_title">"Noch keine Personen vorhanden"</string>
<string name="contact_diary_person_list_no_items_subtitle">"Legen Sie eine Person an und fügen Sie sie in Ihrem Kontakt-Tagebuch hinzu."</string>
<string name="contact_diary_add_person_title">"Person"</string>
<string name="contact_diary_add_person_text_input_name_hint">"Vorname, Nachname"</string>
<string name="contact_diary_add_person_text_input_name_hint">"Name"</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Wasn't it agreed that it should be Vorname, Nachname?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @harambasicluka , checked with Martin Hermes. We believe that 'Name' was agreed upon. Also - this seems to be current implementation in iOS.

@janetback janetback marked this pull request as ready for review March 16, 2021 14:51
@ralfgehrer ralfgehrer changed the title Update contact_diary_strings.xml Minor string fixes (DEV) Mar 16, 2021
@ralfgehrer ralfgehrer self-assigned this Mar 16, 2021
@d4rken d4rken merged commit e3fda1c into release/1.15.x Mar 16, 2021
@d4rken d4rken deleted the fix/string-minor-text-adaptations branch March 16, 2021 18:45
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants