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

Extend RiskLevelStorage (DEV) #2653

Merged
merged 2 commits into from
Mar 19, 2021

Conversation

chiljamgossow
Copy link
Contributor

No description provided.

@chiljamgossow chiljamgossow requested review from a team and BMItr March 19, 2021 14:36
@sonarcloud
Copy link

sonarcloud bot commented Mar 19, 2021

Kudos, SonarCloud Quality Gate passed!

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@BMItr BMItr self-assigned this Mar 19, 2021
@BMItr BMItr added the maintainers Tag pull requests created by maintainers label Mar 19, 2021
@BMItr BMItr added this to the 2.0.0 milestone Mar 19, 2021
Copy link
Contributor

@BMItr BMItr left a comment

Choose a reason for hiding this comment

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

lgtm.
thank you

@@ -36,6 +37,12 @@ interface RiskLevelStorage {
*/
val aggregatedRiskPerDateResults: Flow<List<AggregatedRiskPerDateResult>>

/**
* Risk level per date/day and checkIn
* Used by contact diary overview
Copy link
Contributor

Choose a reason for hiding this comment

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

very minor: "used by" could change easily.

@ralfgehrer ralfgehrer self-assigned this Mar 19, 2021
@chiljamgossow chiljamgossow merged commit ae0a74c into release/2.0.x Mar 19, 2021
@chiljamgossow chiljamgossow deleted the feature/extend-risklevelstorage branch March 19, 2021 15:06
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.

3 participants