Configurações "Suprimir estas tags de emails de resumo" não estão funcionando no email de resumo de prévia?

th21 · Setembro 15, 2023, 11:34pm

Excluí algumas tags nas novas configurações e tentei visualizar o e-mail de resumo, a visualização contém alguns tópicos duplicados e tópicos com essas tags também estão incluídos.

simon · Setembro 16, 2023, 1:21am

Testando isso no meu site de desenvolvimento local, parece ser um bug. O código espera que a configuração digest_suppress_tags seja uma lista de IDs de tags delimitada por pipe, mas na verdade é uma lista de nomes de tags:

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

Veja como a configuração se parece no meu site:

SiteSetting.digest_suppress_tags
=> "foo|bar"

O código que está excluindo as tags do resumo espera que o valor da configuração seja "1|2" (os IDs das tags).

Note que a configuração digest_suppress_categories funciona sem problemas. O seletor de categorias das configurações salva IDs de categorias, não nomes de categorias.

Osama · Setembro 19, 2023, 8:20am

Isso agora está corrigido em:

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.

Obrigado pelo relatório @th21 e pela ótima análise/depuração @simon!

Tópico		Respostas	Visualizações
Suppressed category included in digest Bug activity-summary	4	623	16 de Setembro de 2023
Multiple repeated summary mail entries Bug activity-summary , fixed	30	657	4 de Novembro de 2024
How to show tags in the digest email? Support activity-summary	0	336	23 de Junho de 2021
Digest email settings Support activity-summary	3	1076	14 de Setembro de 2018
Force topics into email digest via tags? Feature activity-summary	1	487	20 de Julho de 2021

Configurações "Suprimir estas tags de emails de resumo" não estão funcionando no email de resumo de prévia?

Tópicos relacionados