「サマリーメールからこれらのタグを抑制する」設定は、プレビューダイジェストメールで機能していませんか？

th21 · 2023 年 9 月 15 日午後 11:34

新しい設定でいくつかのタグを除外しましたが、ダイジェストメールのプレビューを試したところ、プレビューには重複するトピックが含まれており、それらのタグが付いたトピックも含まれていました。

simon · 2023 年 9 月 16 日午前 1:21

ローカル開発サイトでこれをテストしたところ、バグのようです。コードは digest_suppress_tags 設定がパイプで区切られたタグIDのリストであることを期待していますが、実際にはタグ名のリストになっています。

app/models/topic.rb

main


      
          
          unless opts[:include_tl0] || user.user_option.try(:include_tl0_in_digests)
            topics = topics.where("COALESCE(users.trust_level, 0) > 0")
          end
          
          if !!opts[:top_order]
            topics =
              topics.joins("LEFT OUTER JOIN top_topics ON top_topics.topic_id = topics.id").order(<<~SQL)
                COALESCE(topic_users.notification_level, 1) DESC,
                COALESCE(category_users.notification_level, 1) DESC,
                COALESCE(top_topics.#{TopTopic.score_column_for_period(period)}, 0) DESC,
                topics.bumped_at DESC
            SQL
          end
          
          topics = topics.limit(opts[:limit]) if opts[:limit]
          
          # Remove category topics
          topics = topics.where.not(id: Category.select(:topic_id).where.not(topic_id: nil))
          
          # Remove suppressed categories

サイトでの設定は次のようになっています。

SiteSetting.digest_suppress_tags
=> "foo|bar"

ダイジェストからタグを除外するコードは、設定の値が \"1|2\"（タグID）であることを期待しています。

なお、digest_suppress_categories 設定は問題なく動作します。設定のカテゴリセレクターは、カテゴリ名ではなくカテゴリIDを保存します。

Osama · 2023 年 9 月 19 日午前 8:20

これは修正されました。

github.com/discourse/discourse

FIX: Parse the digest_suppress_tags setting correctly (#23623)

committed 07:45AM - 18 Sep 23 UTC

OsamaSayegh

+10 -6

Meta topic: https://meta.discourse.org/t/suppress-these-tags-from-summary-emails…-settings-is-not-working-in-preview-digest-email/279196?u=osama Follow-up to https://github.com/discourse/discourse/commit/477a5dd371e9605a0c0058d0805c767c77fa7c93 The `digest_suppress_tags` setting is designed to be a list of pipe-delimited tag names, but the tag-based topic suppression logic assumes (incorrectly) that the setting contains pipe-delimited tag IDs. This mismatch in expectations led to the setting not working as expected. This PR adds a step that converts the list of tag names in the setting to their corresponding IDs, which is then used to suppress topics tagged with those specific tags.

ご報告いただいた @th21 さん、そして素晴らしい分析とデバッグをしてくださった @simon さん、ありがとうございました！

トピック		返信	表示
Suppressed category included in digest Bug activity-summary	4	623	2023 年 9 月 16 日
Multiple repeated summary mail entries Bug activity-summary , fixed	30	660	2024 年 11 月 4 日
How to show tags in the digest email? Support activity-summary	0	336	2021 年 6 月 23 日
Digest email settings Support activity-summary	3	1077	2018 年 9 月 14 日
Force topics into email digest via tags? Feature activity-summary	1	487	2021 年 7 月 20 日

「サマリーメールからこれらのタグを抑制する」設定は、プレビューダイジェストメールで機能していませんか？

関連トピック