WebHookEventType と解決済みおよび割り当てプラグイン

貢献 バグ
1

この（素晴らしい）変更で、solved プラグインにウェブフックが追加されましたが、ウェブフックイベントタイプに関する一部の機能がコアに移動されました。

ただし、これには望ましくない副作用がいくつかあります。

  1. solved または assign プラグインがインストールされていないインスタンスでは、イベントタイプが表示されますが、翻訳が欠落しており（もちろん）、イベントは利用できません（表示されていますが）。

afbeelding
afbeelding946×439 12.4 KB

  1. solved および assign プラグインは、seedsが欠落しているため、新しい安定版およびベータ版インストールでは現在破損しています。

私の最初の考えは、#2 を修正するために seeds を安定版にバックポートするよう依頼することでしたが、追加のバグ（#1）があるため、シードコードを実際のプラグインに残し、コアのコードで ID を予約する方がよいかもしれません。

長話の要約ですが、以下のコードを solved および assign プラグインに戻す方がよいと思います。

WebHookEventType.seed do |b|
  b.id = WebHookEventType::SOLVED
  b.name = "solved"
end
「いいね！」 8
2

ご指摘ありがとうございます。プラグインがインストールされていない場合にコアに表示されないように、対応できるか検討いたします。

「いいね！」 7
4

ありがとうございます！
安定版でも同様に壊れている点について、対応いただけますでしょうか？

「いいね！」 5
5

この問題に対する修正が作成されました：

#2 については以下の通りです：

最新の discourse-assign および discourse-solved プラグインを使用して、ローカルで stable (2.4.1) を実行していますが、問題は見当たりません。

あなたが遭遇している問題を再現するための情報を提供していただけますか？

「いいね！」 4
6

最新の「solved」プラグインをインストールした安定版（stable）での新規インストールでは、web_hook_event_type テーブルに ID が存在しないため、Webhook タイプが表示されません。

これは、ID のシードデータを作成するコードが「solved」プラグインから削除され、まだ安定版やベータ版に反映されていないことが原因です。

以前からインストールされているサイトでは、古いバージョンの「solved」プラグインを使用していたため、すでにシードデータが存在し、この問題は発生しません。

したがって、再現手順は非常に簡単です：安定版で全く新しいインストールを作成してください。

バージョン 2.4.1 の「solved」プラグイン付きのクリーンな新規インストールでは、Webhook が利用できません。

LKqhCxFVDH
LKqhCxFVDH1182×892 509 KB

手動でシードデータを実行し、再度 Webhook 画面を確認すると表示されます。

pX9fvAw4i7
pX9fvAw4i71084×888 1.01 MB

7

ああ、なるほど。「broken」とは、単に表示されないという意味ですね。

これは仕様によるものです。この機能は 2.5 以降で利用可能です。

「いいね！」 4
8

はい、おっしゃる通りです。不明瞭な説明をしてしまい、申し訳ありません。「壊れている」と「利用できない」を混同していました。これは新機能ですので、実際に問題があるわけではありません。

ただ、プラグイン用のシーディングコードがコアに移動されるのは、少し奇妙に感じられます。ID の予約は理解できますが、シーディングはプラグイン内に残す方がよりクリーンだと考えます。これはあくまで私の個人的な意見ですが :wink:

「いいね！」 3