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

Fix risk card detail screenshots and device time (EXPOSUREAPP-5970) #2688

Merged
merged 3 commits into from
Mar 25, 2021

Conversation

ralfgehrer
Copy link
Contributor

@ralfgehrer ralfgehrer commented Mar 24, 2021

  • The device time is now fixed to 10:00 on all screens.
  • On the risk detail card, with low risk and no exposure, the days since installation shall be visible.
  • On all risk cards, the time of the last successful calculation is now before the device time (at 9:55).

TracingDetailsFragment_tracing_low_risk

TODO:

  • remove the firebase task from circle-ci config after a successful test run

@ralfgehrer ralfgehrer added enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers labels Mar 24, 2021
@ralfgehrer ralfgehrer added this to the 2.0.0 milestone Mar 24, 2021
@ralfgehrer ralfgehrer requested a review from a team March 24, 2021 14:40
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.

LGTM

@sonarcloud
Copy link

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

@harambasicluka harambasicluka self-assigned this Mar 25, 2021
@ralfgehrer ralfgehrer merged commit 0237c53 into release/2.0.x Mar 25, 2021
@ralfgehrer ralfgehrer deleted the fix/5970-screenshots branch March 25, 2021 10:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants