Уведомления по электронной почте о темах не отправляются при добавлении цвета сотрудника к сообщению

zogstrip · 11.Февраль.2026 10:27:00

Better late than never ?!

github.com/discourse/discourse

FIX: Send notification emails for staff-coloured posts (#37712)

main ← fix/staff-coloured-posts-email-notifications

merged 12:26PM - 11 Feb 26 UTC

ZogStriP

+24 -8

When a staff member applies "Staff Colour" to a post, it changes `post_type` fro…m `regular` (1) to `moderator_action` (2). The `NotificationEmailer` only sent emails for `regular` and `whisper` post types, so staff-coloured posts silently skipped email delivery. The same issue affected `PostAlerter`'s `@here` mention expansion which also filtered on post type. This is safe to fix because the only two places that needed updating are `NotificationEmailer::EMAILABLE_POST_TYPES` and `PostAlerter#expand_here_mention`. System-generated moderator posts (close/split/merge) never reach this code — `PostJobsEnqueuer` skips them via `skip_after_create?`, so they never generate the notification types that flow through `NotificationEmailer`. Similarly, system moderator posts don't contain `@here` mentions, so adding `moderator_action` to `expand_here_mention` is equally safe. https://meta.discourse.org/t/370857

Тема		Ответов	Просм.
Can publishing "from" the `#staff` category prevent emails being sent? Support	0	57	01.02.2025
Publication of Timed Topics don't trigger new topic notifications Bug	17	1766	29.06.2020
No email notifications for posts that require approval Bug	9	1837	24.08.2016
Make Staff Notice send a Mailing List email? Support post-notices	4	579	27.08.2023
Notification email for all posts on topic to users who even aren't tracking the topic Support	4	944	09.09.2016

Уведомления по электронной почте о темах не отправляются при добавлении цвета сотрудника к сообщению

Связанные темы