Notifications en direct après Ne pas déranger

Moin · Septembre 7, 2026, 5:51

Merci d’avoir corrigé cela

github.com/discourse/discourse

FIX: Preserve push subscription when DND is active on page load (#43243)

main ← fix/push-notif-unsub-on-dnd

merged 04:34PM - 04 Sep 26 UTC

Alteras1

+1 -1

Previously, loading the site while Do Not Disturb was active would permanently d…estroy the user's push subscription, because `unsubscribePushNotifications` was called unconditionally whenever `isPushNotificationsEnabled` returned false. This change skips the unsubscription when DND is the reason, since the server already suppresses push delivery during DND via `PostAlerter` — the client-side subscription never needed to be torn down.

Sujet		Réponses	Vues
Do not disturb mode Announcements new-feature , pause-notifications	12	5358	Décembre 1, 2023
Android live notifications are now in core! Announcements notifications , new-feature	15	9383	Juin 20, 2018
Can't keep desktop push notifications enabled - dup Bug	8	1569	Février 11, 2021
Desktop Notifications Feature	1	1495	Janvier 5, 2021
Enabling push notifications does not actually subscribe in some cases Bug push-notifications	5	312	Septembre 15, 2025

Notifications en direct après Ne pas déranger

Sujets connexes