uckelman
(Joel Uckelman)
3 Marzo 2024, 2:09pm
1
Sto vedendo questo nella pagina di modifica dei Webhook con 3.3.0.beta1-dev:
Due delle sezioni e le etichette delle loro caselle di controllo sembrano mostrare chiavi i18n invece dei valori di tali chiavi. Ad esempio, sono abbastanza sicuro che
[en.admin.web_hooks.voting_event.group_name]
non sia l’intestazione prevista per qualsiasi categoria che dovrebbe essere.
3 Mi Piace
Ottima osservazione!
Non dovresti vedere il webhook di un plugin non installato sul tuo Discourse.
Questa è una regressione da qui:
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
Nello specifico qui.
I simboli forniti in TYPES non corrispondono ai valori dell’elenco:
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(
Al team, ecco una possibile soluzione:
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 Mi Piace
Grazie per aver segnalato questo problema
L’ho risolto oggi in questo 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 Mi Piace
nat
(Natalie T)
Chiuso
7 Marzo 2024, 4:39am
6
Questo argomento è stato chiuso automaticamente dopo 2 giorni. Non sono più consentite nuove risposte.