Jobs::UserEmail 内の ActionView::Template::Error

zogstrip · 2026 年 2 月 10 日午前 8:02

アハハ！うまいね

複数の返信を1つの通知にまとめ、ユーザー名を上書きしています🤦‍♂️

github.com/discourse/discourse

FIX: Use post author username as fallback in notification emails (#37659)

main ← fix/email-notification-display-username-fallback

opened 08:01AM - 10 Feb 26 UTC

ZogStriP

+23 -1

When multiple replies are collapsed into a single notification, `PostAlerter` se…ts `display_username` to a translated reply count (e.g., "7 réponses") for use in the notification dropdown UI. The `original_username` field always holds the actual poster's username. However, if `original_username` is ever missing from the notification data, `notification_email` falls back to `display_username`, which causes the reply count string to be used as the email sender username. This leads to errors during email rendering. This updates the fallback chain in `notification_email` to prefer `post.user.username` over `display_username`, ensuring a real username is always used even when `original_username` is absent. https://meta.discourse.org/t//395420

トピック		返信	表示
Change Send As in Email Notifications Feature	24	5683	2019 年 1 月 6 日
Get 2 emails for the same reply - user_posted & user_mentioned Bug	3	1105	2016 年 5 月 4 日
Forum topic creator name is used instead of notifier in sent emails with the Notify feature Bug	4	689	2021 年 5 月 26 日
Send rejection email for unrecognised errors Feature	13	1883	2017 年 7 月 25 日
Email notifications reply directly to me (the admin) Support	3	1638	2021 年 6 月 23 日

Jobs::UserEmail 内の ActionView::Template::Error

関連トピック