Live meldingen na niet storen

Moin · 7 september 2026 om 17:51

Thank you fix fixing this

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.

Topic		Antwoorden	Weergaven
Do not disturb mode Announcements new-feature , pause-notifications	12	5283	1 december 2023
Android live notifications are now in core! Announcements notifications , new-feature	15	9281	20 juni 2018
Can't keep desktop push notifications enabled - dup Bug	8	1538	11 februari 2021
Desktop Notifications Feature	1	1477	5 januari 2021
New notification permissions request bar? UX	34	2343	3 oktober 2018

Live meldingen na niet storen

Gerelateerde topics