WebHookEventType e i plugin solved e assign

[RGJ (Richard - Communiteq)]

In questa (fantastica) modifica che ha aggiunto i webhook al plugin solved, alcune funzionalità relative ai tipi di evento webhook sono state spostate nel core.

Tuttavia, questo ha alcuni effetti collaterali indesiderati.

1. Nelle istanze che non hanno installati i plugin solved o assign, i tipi di evento vengono visualizzati ma manca la traduzione e (ovviamente) gli eventi non sono disponibili, sebbene vengano mostrati.

2. I plugin solved e assign sono attualmente rotti nelle nuove installazioni stabili e beta perché mancano i seeds.

Il mio primo pensiero era chiederti di eseguire il backport dei seed nella versione stabile per risolvere il problema #2, ma dato che esiste un bug aggiuntivo (#1), forse è meglio lasciare il codice di seeding nel plugin effettivo e semplicemente riservare gli ID nel codice del core.

In breve, penso che sia meglio spostare il codice

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

di nuovo nei plugin solved e assign.

[blake (Blake Erickson)]

Grazie per averlo sollevato. Controllerò cosa possiamo fare affinché, se i plugin non sono installati, non vengano visualizzati nel core.

[RGJ (Richard - Communiteq)]

Ottimo, grazie!
Potresti per favore provare a risolvere il fatto che è rotto anche nella versione stabile?

[blake (Blake Erickson)]

@RGJ

1. Nelle istanze che non hanno installati i plugin “solved” o “assign”, i tipi di evento vengono visualizzati ma la traduzione manca e, ovviamente, gli eventi non sono disponibili anche se vengono mostrati.

È stata creata una correzione per questo:

FIX: Don't display webhooks for inactive plugins (#9206)

Per quanto riguarda il punto #2:

@RGJ

2. I plugin “solved” e “assign” sono attualmente rotti sulle nuove installazioni stable e beta

Non sto riscontrando problemi nell’eseguire la versione stable (2.4.1) in locale con le ultime versioni dei plugin discourse-assign e discourse-solved.

Potresti fornire alcune informazioni su come posso replicare i problemi che stai riscontrando?

[RGJ (Richard - Communiteq)]

@blake

Potresti fornire alcune informazioni su come posso replicare i problemi che stai riscontrando?

Una nuova installazione su stable con l’ultima versione del plugin solved non avrà gli ID nella tabella web_hook_event_type, impedendo la visualizzazione del tipo di webhook.

Ciò è dovuto al fatto che il codice che popola gli ID è stato rimosso dal plugin solved e non è ancora presente nella versione stable/beta.

I siti già installati che avevano una versione precedente del plugin solved avranno già i dati di seeding e non presenteranno il problema.

Quindi, il passaggio per la replica è semplice: crea una nuova installazione su stable.

Installazione nuova e pulita su 2.4.1 con plugin solved: nessun webhook disponibile.

Eseguiamo manualmente il seeding, poi controlliamo di nuovo la schermata dei webhook e apparirà:

[blake (Blake Erickson)]

Oh, ok, quindi “rotto” significa che semplicemente non viene visualizzato.

@RGJ

Installazione nuova e pulita su 2.4.1 con il plugin risolto: nessun webhook disponibile.

Questo è previsto dal progetto. È una funzionalità disponibile dalla versione 2.5 in poi.

[RGJ (Richard - Communiteq)]

Ok, hai perfettamente ragione, mi scuso per non essere stato chiaro. Stavo confondendo ‘rotto’ con ‘non disponibile’, e dato che si tratta di una nuova funzionalità, in effetti non c’è nulla di rotto.

Tuttavia, mi sembra un po’ strano che il codice di seeding per un plugin venga spostato nel core. Capisco che tu voglia riservare gli ID, ma penso che sarebbe più pulito lasciare il seeding nel plugin. Ma questa è solo la mia opinione personale