¿La configuración "Ocultar estas etiquetas de los correos electrónicos de resumen" no funciona en el correo electrónico de resumen de vista previa?

th21 · 15 Septiembre, 2023 23:34

He excluido algunas etiquetas en la nueva configuración e intento previsualizar el correo electrónico de resumen, la vista previa contiene algunos temas duplicados y también se incluyen temas con esas etiquetas.

simon · 16 Septiembre, 2023 01:21

Probando esto en mi sitio de desarrollo local, parece ser un error. El código espera que la configuración digest_suppress_tags sea una lista de IDs de etiquetas delimitada por barras verticales, pero en realidad es una lista de nombres de etiquetas:

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

Así es como se ve la configuración en mi sitio:

SiteSetting.digest_suppress_tags
=> "foo|bar"

El código que está excluyendo las etiquetas del resumen espera que el valor de la configuración sea "1|2" (los IDs de las etiquetas).

Tenga en cuenta que la configuración digest_suppress_categories funciona sin ningún problema. El selector de categorías de la configuración guarda IDs de categorías, no nombres de categorías.

Osama · 19 Septiembre, 2023 08:20

Esto ahora está arreglado en:

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.

¡Gracias por el informe @th21 y por el gran análisis/depuración @simon!

Tema		Respuestas	Vistas
Suppressed category included in digest Bug activity-summary	4	623	16 Septiembre 2023
Multiple repeated summary mail entries Bug activity-summary , fixed	30	660	4 Noviembre 2024
How to show tags in the digest email? Support activity-summary	0	336	23 Junio 2021
Digest email settings Support activity-summary	3	1077	14 Septiembre 2018
Force topics into email digest via tags? Feature activity-summary	1	487	20 Julio 2021

¿La configuración "Ocultar estas etiquetas de los correos electrónicos de resumen" no funciona en el correo electrónico de resumen de vista previa?

Temas relacionados