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

Change screenshot data for contact diary (DEV) #2584

Merged

Conversation

LukasLechnerDev
Copy link
Contributor

@LukasLechnerDev LukasLechnerDev commented Mar 11, 2021

Our screenshots should have the following texts for the contact diary:

  • "Rewe" -> "Supermarkt"
  • "Andrea Steinhauer" -> "Erika Musterfrau"
  • "Constantin Frenzel" -> "Max Mustermann"

The pipeline will run firebase_screenshots, so the screenshots are an artifact in CircleCI. You can take a look there to verify if the correct texts are displayed.

Todo before merging:

  • Revert last commit to disable firebase_screenshot again

@LukasLechnerDev LukasLechnerDev added maintainers Tag pull requests created by maintainers author merge PR to be merged by author labels Mar 11, 2021
@LukasLechnerDev LukasLechnerDev added this to the 1.15.0 milestone Mar 11, 2021
@LukasLechnerDev LukasLechnerDev requested a review from a team March 11, 2021 16:35
harambasicluka
harambasicluka previously approved these changes Mar 11, 2021
@harambasicluka harambasicluka self-assigned this Mar 11, 2021
@ralfgehrer ralfgehrer self-assigned this Mar 11, 2021
Copy link
Contributor

@ralfgehrer ralfgehrer left a comment

Choose a reason for hiding this comment

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

All good once the Todo is done.

@ralfgehrer ralfgehrer removed the author merge PR to be merged by author label Mar 11, 2021
@sonarcloud
Copy link

sonarcloud bot commented Mar 11, 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

@ralfgehrer ralfgehrer merged commit 5b7105e into release/1.15.x Mar 11, 2021
@ralfgehrer ralfgehrer deleted the fix/DEV-Change-screenshot-data-for-contact-diary branch March 11, 2021 18:13
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

3 participants