Bug 475390

Summary: Do not disturb blocks Francis notifications
Product: [Applications] Francis Reporter: Quinten Kock <quinten>
Component: GeneralAssignee: Thomas Duckworth <tduck>
Status: RESOLVED FIXED    
Severity: normal CC: tduck
Priority: NOR    
Version First Reported In: 1.0.1   
Target Milestone: ---   
Platform: Flatpak   
OS: Linux   
Latest Commit: https://invent.kde.org/utilities/francis/-/commit/b240c938644058c1c52123e314e1e2ef91b79d78 Version Fixed/Implemented In:
Sentry Crash Report:

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