nucleus-ffm / foss_warn

An unofficial open source application to get emergency alerts from https://warnung.bund.de/meldungen.
GNU General Public License v3.0
100 stars 6 forks source link

App ignore do not disturb #106

Closed Tealk closed 10 months ago

Tealk commented 11 months ago

Describe the bug For me, the notifications ignore the "do not disturb" setting

To Reproduce Steps to reproduce the behavior:

  1. Wait for a notification

Expected behavior Only the exceptions on the "do not disturb" list should be audible

**Device :** - Device: Pixel 6 - OS: GraphenOS - App-version 0.6.1 - Battery optimization false
nucleus-ffm commented 11 months ago

Thank you for your issue. This behavior is not really a bug, but we need to discuss if we want a different behavior. We use the notification category Alarm for alerts, which in default mode is an exception in the Do Not Disturb settings. If you disable Alarm as an exception in the Do Not Disturb-settings, the FOSSWarns notifications will be muted. The question is whether we want that. We could change the category to something like Message so that the user can still enable the Do Not Disturb setting for some notification channels. @MatsG23 what do you think?

MatsG23 commented 11 months ago

On the one hand, I think that the Alarm category was made to be used by timer/clock apps and should not be used by other apps. On the hand, it would be nice if, at least, warnings with a severe or extreme severity could bypass the DND mode (by default). Can the app set the "Can bypass DND mode" setting itself or is this an user-only option? I understand that some warnings (hopefully most) are considered quite unimportant and a distraction from e.g. the workflow tried to achieve through the DND mode.

Tealk commented 11 months ago

Wouldn't it make sense to simply use normal notifications? Everyone can then decide for themselves in the settings whether Do Not Disturb should be bypassed or not.

e.g. the workflow tried to achieve through the DND mode.

The trigger for the issue was that I was constantly getting alerts even though I was asleep.

nucleus-ffm commented 10 months ago

We have changed the category to Message in version 0.7.0, which should solve this issue.