WebHookEventType und die gelösten und zugewiesenen Plugins

Beitragen Fehler
1

In dieser (fantastischen) Änderung, die Webhooks zum „solved“-Plugin hinzufügte, wurde ein Teil der Funktionalität bezüglich der Webhook-Ereignistypen in den Kern verlegt.

Dies hat jedoch einige unerwünschte Nebeneffekte.

  1. Auf Instanzen, auf denen die Plugins „solved“ oder „assign“ nicht installiert sind, werden die Ereignistypen angezeigt, aber die Übersetzung fehlt, und (natürlich) sind die Ereignisse nicht verfügbar, obwohl sie angezeigt werden.

afbeelding
afbeelding946×439 12.4 KB

  1. Die Plugins „solved“ und „assign“ sind auf neuen stabilen und Beta-Installationen derzeit defekt, da ihnen die Seeds fehlen.

Mein erster Gedanke war, Sie zu bitten, die Seeds auf die stabile Version zurückzuportieren, um Problem #2 zu beheben. Da jedoch ein zusätzlicher Fehler (#1) besteht, wäre es vielleicht besser, den Seed-Code im eigentlichen Plugin zu belassen und lediglich die IDs im Kern-Code zu reservieren.

Kurz gesagt: Ich denke, es wäre besser, den folgenden Code

WebHookEventType.seed do |b|
  b.id = WebHookEventType::SOLVED
  b.name = "solved"
end

wieder in die Plugins „solved“ und „assign“ zu verlegen.

8 „Gefällt mir“
2

Danke, dass du das ansprichst. Ich werde prüfen, was wir tun können, damit Plugins, die nicht installiert sind, nicht im Core angezeigt werden.

7 „Gefällt mir“
4

Super, danke!
Könntest du bitte auch das Problem beheben, dass es auch in der stabilen Version nicht funktioniert?

5 „Gefällt mir“
5

Eine Lösung dafür wurde erstellt:

Was Punkt 2 betrifft:

Ich sehe keine Probleme beim lokalen Betrieb von Version 2.4.1 (stabil) mit den neuesten Versionen der Plugins „discourse-assign“ und „discourse-solved“.

Könntest du mir bitte einige Informationen geben, damit ich die von dir beschriebenen Probleme nachstellen kann?

4 „Gefällt mir“
6

Eine neue Installation auf der stabilen Version mit der neuesten Version des „solved

7

Ah okay, also bedeutet „broken

4 „Gefällt mir“
8

Okay, du hast völlig recht, entschuldige bitte, dass ich unklar war. Ich habe ‘kaputt’ und ‘nicht verfügbar’ verwechselt, und da es sich um ein neues Feature handelt, ist tatsächlich nichts kaputt.

Es fühlt sich trotzdem etwas seltsam an, dass der Seed-Code für ein Plugin in den Core verschoben wurde. Ich verstehe, dass du die IDs reservieren möchtest, aber ich finde, es wäre sauberer, den Seed im Plugin zu belassen. Das ist aber nur meine persönliche Meinung :wink:

3 „Gefällt mir“