Bug 475390 - Do not disturb blocks Francis notifications
Summary: Do not disturb blocks Francis notifications
Status: RESOLVED FIXED
Alias: None
Product: Francis
Classification: Applications
Component: General (show other bugs)
Version: 1.0.1
Platform: Flatpak Linux
: NOR normal
Target Milestone: ---
Assignee: Thomas Duckworth
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-09 03:26 UTC by Quinten Kock
Modified: 2024-05-17 13:29 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Quinten Kock 2023-10-09 03:26:13 UTC
SUMMARY
When working, I tend to put my system in Do Not Disturb mode, so that I don't get distracted as easily. It seems that by default this however also blocks the Francis notification, which kind of defeats the point.

STEPS TO REPRODUCE
1. Turn on do not disturb 
2. Start Francis
3. Wait for the timer to elapse (is there a way to change it so that I can test this more easily?) 

OBSERVED RESULT
Notification gets blocked

EXPECTED RESULT
Notification goes through

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: Arch Linux
(available in About System)
KDE Plasma Version: 5.27.8
KDE Frameworks Version: 5.110.0
Qt Version: 5.15.10
Francis 1.0.1 from Flatpak

ADDITIONAL INFORMATION
Looking at the code, it seems Francis sends a low priority notification. Perhaps this should be marked critical instead, so that it overrides DnD?
Also, I think the notification goes through the "Plasma Workspace" app, so enabling notifications for that in DnD might be a viable workaround? It might be better for it to be a separate app so that, worst case, letting the user toggle this would be more obvious.
Comment 1 Thomas Duckworth 2024-05-12 09:49:43 UTC
I'm currently working on adding an option to the top menubar to do this. It sets the notification priority to Critical, so it goes through in Do Not Disturb.
Comment 2 Thomas Duckworth 2024-05-12 10:05:27 UTC
Opened a MR https://invent.kde.org/utilities/francis/-/merge_requests/16
Comment 3 Carl Schwan 2024-05-17 13:29:06 UTC
Git commit b240c938644058c1c52123e314e1e2ef91b79d78 by Carl Schwan, on behalf of Thomas Duckworth.
Committed on 17/05/2024 at 13:25.
Pushed by carlschwan into branch 'master'.
Add checkbox in PageHeader.qml to bypass Do not disturb, setting the notification urgency to Critical

M  +4    -0    src/config.kcfg
M  +19   -0    src/ui/PageHeader.qml
M  +2    -2    src/ui/main.qml

https://invent.kde.org/utilities/francis/-/commit/b240c938644058c1c52123e314e1e2ef91b79d78