corona-warn-app / cwa-app-ccl

Common Covid Logic (CCL) for Android and iOS
Apache License 2.0
7 stars 4 forks source link

Chore: consolidate CCL release 3.3.x #105

Closed mlenkeit closed 2 years ago

mlenkeit commented 2 years ago

This PR consolidates the changes for CCL 3.3.x in a separate branch, e.g. to leverage the OS-specific integration tests and to consolidate the UA review.

Ein-Tim commented 2 years ago

I have a question reg. https://github.com/corona-warn-app/cwa-app-ccl/pull/107, if the texts & the immunization state in the app changes to partial immunization only, will there be a booster recommendation notification which not only recommends the booster but clearly states that a booster is necessary now?

mlenkeit commented 2 years ago

I have a question reg. #107, if the texts & the immunization state in the app changes to partial immunization only, will there be a booster recommendation notification which not only recommends the booster but clearly states that a booster is necessary now?

@Ein-Tim this is not planned at the moment.

Ein-Tim commented 2 years ago

@mlenkeit

So the user will just loose their fully vaccinated status without further information in the app why?

I think more information should be given.

MikeMcC399 commented 2 years ago

@Ein-Tim

So the user will just loose their fully vaccinated status without further information in the app why?

I think more information should be given.

This is more the responsibility of the new features text in the app e.g. for Android 2.27.x https://github.com/corona-warn-app/cwa-app-android/blob/release/2.27.x/Corona-Warn-App/src/main/res/values/release_info_strings.xml.

At the moment only the mask rule change is mentioned in the release info. It would be good for the release info to also explain that the 2G, 2G+, 3G and 3G+ status is removed and that the rules in general have been updated to correspond to current regulations.

Also the blog text for the release 2.27 would have the opportunity to explain about this.

This is all work-in-progress, so I would wait and see what develops.

Ein-Tim commented 2 years ago

@MikeMcC399

I'll take your advice and look at it again in a week or so.

Ein-Tim commented 2 years ago

Okay, question answered with https://github.com/corona-warn-app/dcc-rule-translation/issues/10#issuecomment-1252418923.