Le impostazioni "Suppress these tags from summary emails" non funzionano nell'email di riepilogo in anteprima?

th21 · 15 Settembre 2023, 11:34pm

Ho escluso alcuni tag nelle nuove impostazioni e provo ad anteprima l’email di riepilogo, l’anteprima contiene alcuni argomenti duplicati e sono inclusi anche argomenti con quei tag.

simon · 16 Settembre 2023, 1:21am

Sto testando questo sul mio sito di sviluppo locale, sembra essere un bug. Il codice si aspetta che l’impostazione digest_suppress_tags sia un elenco delimitato da barre verticali di ID di tag, ma in realtà è un elenco di nomi di tag:

github.com/discourse/discourse

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

Ecco come appare l’impostazione sul mio sito:

SiteSetting.digest_suppress_tags
=> "foo|bar"

Il codice che esclude i tag dal digest si aspetta che il valore dell’impostazione sia "1|2" (gli ID dei tag).

Nota che l’impostazione digest_suppress_categories funziona senza problemi. Il selettore di categorie delle impostazioni salva gli ID delle categorie, non i nomi delle categorie.

Osama · 19 Settembre 2023, 8:20am

Questo è ora corretto in:

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.

Grazie per la segnalazione @th21 e per la grande analisi/debug @simon!

Argomento		Risposte	Visualizzazioni
Suppressed category included in digest Bug activity-summary	4	623	Settembre 16, 2023
Multiple repeated summary mail entries Bug activity-summary , fixed	30	660	Novembre 4, 2024
How to show tags in the digest email? Support activity-summary	0	336	Giugno 23, 2021
Digest email settings Support activity-summary	3	1077	Settembre 14, 2018
Force topics into email digest via tags? Feature activity-summary	1	487	Luglio 20, 2021

Le impostazioni "Suppress these tags from summary emails" non funzionano nell'email di riepilogo in anteprima?

Argomenti correlati