uckelman
(Joel Uckelman)
Mars 3, 2024, 2:09
1
Je vois ceci sur la page d’édition des Webhooks avec 3.3.0.beta1-dev :
Deux des sections et les étiquettes de leurs cases à cocher semblent afficher des clés i18n au lieu des valeurs de ces clés. Par exemple, je suis à peu près sûr que
[en.admin.web_hooks.voting_event.group_name]
n’est pas l’en-tête prévu pour la catégorie que cela est censé être.
3 « J'aime »
Bonne remarque !
Vous ne devriez pas voir le webhook d’un plugin qui n’est pas installé sur votre Discourse.
C’est une régression à partir d’ici :
committed 03:35AM - 09 Oct 23 UTC
Before this change, webhooks could be only configured for specific groups like f… or example, all topic events.
We would like to have more granular control like for example topic_created or topic_destroyed.
Test are failing because plugins changed has to be merged as well:
discourse/discourse-assign#498
discourse/discourse-solved#248
discourse/discourse-topic-voting#159
Spécifiquement ici.
Les symboles fournis dans TYPES ne correspondent pas aux valeurs de la liste :
reviewable_updated: 902,
notification_created: 1001,
solved_accepted_solution: 1101,
solved_unaccepted_solution: 1102,
scope :active_grouped, -> { active.where.not(group: nil).group_by(&:group) }
def self.active
ids_to_exclude = []
unless defined?(SiteSetting.solved_enabled) && SiteSetting.solved_enabled
ids_to_exclude.concat([TYPES[:solved_accepted_solution], TYPES[:solved_unaccepted_solution]])
end
unless defined?(SiteSetting.assign_enabled) && SiteSetting.assign_enabled
ids_to_exclude.concat([TYPES[:assign_assigned], TYPES[:assign_unassigned]])
end
unless defined?(SiteSetting.topic_voting_enabled) && SiteSetting.topic_voting_enabled
ids_to_exclude.concat([TYPES[:voting_topic_upvote], TYPES[:voting_topic_unvote]])
end
unless defined?(SiteSetting.chat_enabled) && SiteSetting.chat_enabled
ids_to_exclude.concat(
À l'équipe, voici une solution possible :
def self.types_with_prefix(prefix)
TYPES.select { |key, _value| key.to_s.start_with?(prefix.to_s) }.values
end
def self.active
ids_to_exclude = []
unless defined?(SiteSetting.solved_enabled) && SiteSetting.solved_enabled
ids_to_exclude.concat(types_with_prefix(:solved))
end
unless defined?(SiteSetting.assign_enabled) && SiteSetting.assign_enabled
ids_to_exclude.concat(types_with_prefix(:assign))
end
unless defined?(SiteSetting.voting_enabled) && SiteSetting.voting_enabled
ids_to_exclude.concat(types_with_prefix(:voting_topic))
end
unless defined?(SiteSetting.chat_enabled) && SiteSetting.chat_enabled
ids_to_exclude.concat(types_with_prefix(:chat_message))
end
self.where.not(id: ids_to_exclude.flatten)
end
6 « J'aime »
Merci d’avoir signalé ce problème
Je l’ai corrigé aujourd’hui dans cette PR :
main ← lis2:active-webhooks
opened 11:28PM - 04 Mar 24 UTC
Bug introduced when webhooks were granularized in this PR - https://github.com/d… iscourse/discourse/commit/c46811092994d456e7a67b33669b76e51aaa3668
Only active webhooks should be available when webhooks are configured.
https://meta.discourse.org/t/i18n-keys-showing-on-webhooks-edit-page/297701
5 « J'aime »
nat
(Natalie T)
A fermé ce sujet ()
Mars 7, 2024, 4:39
6
Ce sujet a été automatiquement fermé après 2 jours. Les nouvelles réponses ne sont plus autorisées.