-
Notifications
You must be signed in to change notification settings - Fork 343
"Exposures with low risk on n day(s)" vs "Exposures on n day(s) with increased risk" #578
Comments
@Ein-Tim Thanks. Internal Tracking ID: EXPOSUREAPP-6264 |
I provided PR corona-warn-app/cwa-app-ios#4458 addressing this issue. |
After I provided PR corona-warn-app/cwa-app-ios#4458 @dsarkar informed me that this is a wont fix. I understand that and this is ok for me, still I'm unhappy about the communication between the team and the community here, see corona-warn-app/cwa-app-ios#4458 (comment). Closing. |
Hi @Ein-Tim, suggestion: in order to avoid working on a PR that might be rejected, you can ping me or another Corona-Warn-App Open Source Team member and we will try to get feedback from the dev team. And as usual, many thanks for contributing here in this project! |
Hey @dsarkar, yes, I will do that in the future, however, issues should not only be rejected when someone wants to work on it but continuously, so that the issue trackers are kept clean and so that the community knows that open issues are valid and can be worked on / are worked on by the team. Do you agree with me here? |
Hi @Ein-Tim, sure we try to sync as much as possible between the Community and dev team/management/etc... Some issues unfortunately snow under, and only when there is a comment, or in this case a PR, they "awake again to life". This happens particularly with non-critical issues. |
@Ein-Tim for what it's worth, let me elaborate on the wording:
This makes sense, because the risk calculation (1) first distinguishes between exposures as being either low risk or increased risk and then it aggregates the data by date to decide if the day has a low risk or increased risk. That means there can be a day with increased risk, while all exposures on that day were actually low risk. This would overlap with Exposures on n day(s) with low risk and exclude Exposures with increased risk on n day(s). That's why we have the current wording. Apologies for not explaining this earlier. |
@mlenkeit Thank you very much for the explanation! This makes sense!
That's absolutely no problem, I understand that the team has a huge workload and that it is nearly impossible to keep up with everything! Have a relaxed evening & weekend! |
Where to find the issue
Inside of the app on the main risk card, see "Steps to reproduce the issue".
Steps to reproduce the issue
Describe the issue
The app does not display the number of encounters in the same place for an increased risk encounter (middle of the sentence) as it does for a low risk encounter (end of the sentence).
Suggested change
Let the app display the number of encounters for both, low and increased risk, in the same place of the sentence (either in the middle or on the end).
So either
"Exposures with low risk on n day(s)" &
"Exposures with increased risk on n day(s)"
or
"Exposures on n day(s) with low risk" &
"Exposures on n day(s) with increased risk"
Screenshots
See https://www.coronawarn.app/en/screenshots.
Internal Tracking ID: EXPOSUREAPP-6264
The text was updated successfully, but these errors were encountered: