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

Change term #4760

Merged
merged 2 commits into from
Feb 1, 2022
Merged

Change term #4760

merged 2 commits into from
Feb 1, 2022

Conversation

CV113
Copy link
Contributor

@CV113 CV113 commented Feb 1, 2022

RKI has changed the term they use on the page Digitales Impfquotenmonitoring zur COVID-19-Impfung. Now they are using the term "Grundimmunisiert" where a couple of days ago the graph was labelled "vollständig geimpft".

--> https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-11520

@CV113
Change term
41a5f66 
RKI has changed the term they use on the page Digitales Impfquotenmonitoring zur COVID-19-Impfung. Now they are using the term "Grundimmunisiert" where a couple of days ago the graph was labelled "vollständig geimpft".

--> https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-11520
@CV113 CV113 added the text change PRs with text changes. label Feb 1, 2022
@CV113 CV113 added this to the 2.18.0 milestone Feb 1, 2022
@CV113 CV113 requested review from KathrinLuecke and a team February 1, 2022 10:06
@MikeMcC399
Copy link
Contributor

@MikeMcC399 MikeMcC399 mentioned this pull request Feb 1, 2022
Closed
3 tasks
KathrinLuecke
KathrinLuecke approved these changes Feb 1, 2022
View reviewed changes
Copy link
Contributor

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

@marcauberer marcauberer self-assigned this Feb 1, 2022
@sonarcloud
Copy link

sonarcloud bot commented Feb 1, 2022

Kudos, SonarCloud Quality Gate passed!    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

@marcauberer marcauberer merged commit df53c5c into release/2.18.x Feb 1, 2022
@marcauberer marcauberer deleted the CV113-patch-1-Change-term branch February 1, 2022 11:03
@MikeMcC399
Copy link
Contributor

@marcauberer / @CV113

You still refer to "vollständig geimpft" in

cwa-app-android/Corona-Warn-App/src/main/res/values-de/release_info_strings.xml

Line 30 in df53c5c

<item>Neben den Statistiken für mindestens einmal geimpfte und vollständig geimpfte Personen, finden Sie jetzt auch die Anzahl der Personen, die eine Auffrischimpfung erhalten haben.</item>

As far as statistics is concerned, this should also be changed to use the term "grundimmunisiert".

@CV113
Copy link
Contributor Author

CV113 commented Feb 1, 2022

@MikeMcC399 I changed this too :-)

@MikeMcC399
Copy link
Contributor

@CV113

I changed this too :-)

Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@KathrinLuecke KathrinLuecke KathrinLuecke approved these changes

@marcauberer marcauberer marcauberer approved these changes

Assignees

@marcauberer marcauberer

Labels
text change PRs with text changes.
Projects
None yet
Milestone
2.18.0
Development

Successfully merging this pull request may close these issues.
4 participants
@CV113 @MikeMcC399 @marcauberer @KathrinLuecke