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

Certificates info screen update (EXPOSUREAPP-7661) #3450

Merged

Conversation

AlexanderAlferov
Copy link
Contributor

Adapted certificates info screen (with new legal strings 🤩).

Screenshot 2021-06-15 at 15 32 38

@AlexanderAlferov AlexanderAlferov added ui Issue related to UI aspects maintainers Tag pull requests created by maintainers text change PRs with text changes. labels Jun 15, 2021
@AlexanderAlferov AlexanderAlferov added this to the 2.5.0 milestone Jun 15, 2021
@AlexanderAlferov AlexanderAlferov marked this pull request as ready for review June 15, 2021 13:16
@AlexanderAlferov AlexanderAlferov requested review from a team June 15, 2021 13:16
@axelherbstreith axelherbstreith self-assigned this Jun 15, 2021
@mtwalli mtwalli self-assigned this Jun 15, 2021
@mtwalli
Copy link
Contributor

mtwalli commented Jun 15, 2021

the text in iOS is different, could you please check that ? and which one is up to date ?

Copy link
Contributor

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

@axelherbstreith axelherbstreith left a comment

Choose a reason for hiding this comment

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

lgtm - Darkmode Icons are also in place 👍

@mtwalli mtwalli added the author merge PR to be merged by author label Jun 15, 2021
…-certificates-info-screen-update

# Conflicts:
#	Corona-Warn-App/src/main/res/values-de/vaccination_strings.xml
Copy link
Contributor

@mtwalli mtwalli left a comment

Choose a reason for hiding this comment

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

👍🏼 Thanks!

@sonarcloud
Copy link

sonarcloud bot commented Jun 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

Copy link
Contributor

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
author merge PR to be merged by author maintainers Tag pull requests created by maintainers text change PRs with text changes. ui Issue related to UI aspects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants