-
Notifications
You must be signed in to change notification settings - Fork 495
Enhancement to notification of risk change #1066
Comments
The data privacy has strictly standards to not show any push notifications where other people can see the app is installed or the person received a test result. If it's really necessary the notification must not show sensible information or any hints that the person did a test. |
@thomasaugsten thanks for the information, then I'll limit this enhancement idea just to a suggestion to add any sort of title, maybe same colorful logo as in the app, and allow the notification to be expanded to display the full text (by "dragging down" on it), right now user can just read:
instead of:
|
Looks good feel free to open a PR for this enhancement |
@thomasaugsten I've opened PR only for expandable text as changing the title would require changes in strings for all the languages, but the functionality to add the title is in the code already, it's just defined as "" (same line as the line affected by PR) |
Avoid duplicates
Current Implementation
Currently the notification does not have any title, and it cannot be expanded to view the full text
Suggested Enhancement
It could be displayed similar to notification which can be generated by ENF when querying
ExposureInformation
, which has a title and full text visible. Additionally it could include a message about the risk directly in the notification. Would be nice to also have the icon in the same color as original CWA logo, not grey, to make it a bit more prominent.Expected Benefits
Improved UX, better chance that notification is not dismissed accidentally.
The text was updated successfully, but these errors were encountered: