Fehler in Bezug auf Pings

Falco · 30. April 2026 um 19:05

In folgendem Pull Request behoben:

github.com/discourse/discourse

FIX: Duplicate notifications when mentioned in chat (#39670)

main ← fix-chat-mention-duplicate-notifications

merged 05:52PM - 30 Apr 26 UTC

xfalcox

+206 -0

Previously, building the web push "Reply" action payload raised because `inline_…reply.png` lived under `public/images/` where Propshaft can't resolve it, so `Jobs::Chat::NotifyMentioned` retried after `create_notification!` and inflated the bell with duplicate rows. This change moves the icon into `app/assets/images/push-notifications/` and wraps the action builder in a `rescue` so any future payload failure logs and returns `{}` instead of stranding the job. A post-deploy migration sweeps the duplicate `chat_mention` `Notification` rows (and their `chat_mention_notifications` join entries) created during the deploy window — keeping the oldest row per `(user_id, data)` group and merging the read state forward.

Thema		Antworten	Aufrufe
Chat messages not showing up unless I refresh or switch channels Bug chat	9	1684	21. Juli 2024
Bug with chat notifications not showing up Bug notifications , chat	26	987	25. November 2025
Disable @all pings from Discourse Chat Feature chat	20	954	15. März 2023
Error occurred and wrong number of replies? Support	4	1057	15. April 2016
Moderator actions are invisible in mail notifications Bug	5	1284	26. Februar 2016

Fehler in Bezug auf Pings

Verwandte Themen