Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated days of risk calculation #2691

Merged
merged 5 commits into from
Apr 6, 2022
Merged

updated days of risk calculation #2691

merged 5 commits into from
Apr 6, 2022

Conversation

amouzayanni
Copy link
Contributor

Change 14 to 10

Co-Authored-By: amouzayanni <[email protected]>
@amouzayanni amouzayanni requested a review from a team April 5, 2022 20:29
src/data/faq.json Outdated Show resolved Hide resolved
src/data/faq.json Outdated Show resolved Hide resolved
Copy link
Contributor

@MikeMcC399 MikeMcC399 left a comment

Choose a reason for hiding this comment

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

See in-line comments

@amouzayanni
Copy link
Contributor Author

Just made the suggested changes. Please let me know if there are any issues.

Copy link
Contributor

@MikeMcC399 MikeMcC399 left a comment

Choose a reason for hiding this comment

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

I have no more change requests for the text.

Please mark the reviews as resolved.

Copy link
Contributor

@Ein-Tim Ein-Tim left a comment

Choose a reason for hiding this comment

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

LGTM!

@dsarkar
Copy link
Member

dsarkar commented Apr 6, 2022

Many thanks to @amouzayanni @Ein-Tim and @MikeMcC399 ! @amouzayanni Congrats to your first PR in this project!

@dsarkar dsarkar merged commit 5063def into corona-warn-app:master Apr 6, 2022
@amouzayanni amouzayanni deleted the fix-days-of-risk branch April 6, 2022 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants