Funktion "Diese Tags von Zusammenfassungs-E-Mails unterdrücken" funktioniert nicht in der Vorschau-Digest-E-Mail?

th21 · 15. September 2023 um 23:34

Ich habe einige Tags in den neuen Einstellungen ausgeschlossen und versuche, die Digest-E-Mail in der Vorschau anzuzeigen. Die Vorschau enthält einige doppelte Themen, und Themen mit diesen Tags sind ebenfalls enthalten.

simon · 16. September 2023 um 01:21

Ich teste dies auf meiner lokalen Entwicklungsseite, es scheint ein Fehler zu sein. Der Code erwartet, dass die Einstellung digest_suppress_tags eine Pipe-getrennte Liste von Tag-IDs ist, aber es ist tatsächlich eine Liste von Tag-Namen:

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

So sieht die Einstellung auf meiner Seite aus:

SiteSetting.digest_suppress_tags
=> "foo|bar"

Der Code, der die Tags aus der Zusammenfassung ausschließt, erwartet, dass der Wert der Einstellung "1|2" (die Tag-IDs) ist.

Beachten Sie, dass die Einstellung digest_suppress_categories ohne Probleme funktioniert. Der Einstellungskategorie-Selektor speichert Kategorie-IDs, nicht Kategorienamen.

Osama · 19. September 2023 um 08:20

Dies ist jetzt behoben 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.

Vielen Dank für den Bericht @th21 und für die großartige Analyse/Fehlerbehebung @simon!

Thema		Antworten	Aufrufe
Suppressed category included in digest Bug activity-summary	4	623	16. September 2023
Multiple repeated summary mail entries Bug activity-summary , fixed	30	657	4. November 2024
How to show tags in the digest email? Support activity-summary	0	336	23. Juni 2021
Digest email settings Support activity-summary	3	1076	14. September 2018
Force topics into email digest via tags? Feature activity-summary	1	487	20. Juli 2021

Funktion "Diese Tags von Zusammenfassungs-E-Mails unterdrücken" funktioniert nicht in der Vorschau-Digest-E-Mail?

Verwandte Themen