Risposta Discourse

Il nostro pulsante “reazione” non funziona dopo l’ultimo aggiornamento.

Non riesco a trovare alcun errore nella Console mentre clicco sull’icona emoji d-icon. Con le mie conoscenze limitate su come è implementato, non riesco a individuare il bug. Spero che qualcuno riesca a :-D… (Ho provato a esaminare il repository, ma non ho notato nulla, con le mie conoscenze limitate sulla creazione di plugin per Discourse (!))

3 Mi Piace

Stesso problema. Non riesco a cliccare sul mio sito web.

2 Mi Piace

Posso confermare che questo bug esiste davvero. Dopo aver cliccato, non succede nulla.

2 Mi Piace

Posso confermare anch’io. Quando clicco sull’icona delle reazioni per impostarla, non appare nulla, non succede niente…

2 Mi Piace

Aggiungo #plugin:broken-plugin qui per il momento. Fammi sapere quando sarà risolto.

2 Mi Piace

Qualcuno ha scoperto quale potrebbe essere il problema? Non ci sono state modifiche nel codice del plugin da un bel po’ di tempo, quindi immagino che qualcosa nel codice di Discourse sia cambiato, causando questo problema (anche se non ho notato modifiche evidenti che potrebbero influenzarlo…).

1 Mi Piace

Sono disponibili più aggiornamenti di Discourse che devono essere gestiti in questo plugin.

Ho creato una PR che può servire come soluzione temporanea. Probabilmente sarò condannato alle fiamme degli inferi di Tartaro per il modo in cui funziona la correzione, ma potrebbe fungere da misura provvisoria @gdpelican

@Jite Sembri interessato. Vorresti imparare qualcosa in più sullo sviluppo di Discourse e sul funzionamento di questo plugin per contribuire alla sua manutenzione? Posso aiutarti a conoscere i dettagli del codice di Discourse.

8 Mi Piace

Mi sarebbe piaciuto moltissimo accettare la tua offerta, ma al momento ho già troppo da fare per poterlo fare. Ho già creato alcuni plugin più piccoli, ma la mia esperienza con Ruby è così limitata che non ho ancora avuto la possibilità di approfondire troppo gli interni, hehe.
Rimarrò comunque qui, quindi se troverò più tempo per farlo, sarò felice di farti sapere! :slight_smile:

Darò anche un’occhiata alla correzione, perché l’istanza su cui uso il plugin sembra proprio piacergli :smiley:

4 Mi Piace

Inoltre, mi chiedo davvero come non abbia trovato quella merge nei log, li ho letti come un libro ieri! È stata una mia negligenza ;D

3 Mi Piace

Fammi sapere :slight_smile: Sarò felice di aiutarti a prendere confidenza con Discourse.

5 Mi Piace

:sob:
Adoro questo plugin, vorrei che fosse mantenuto. In realtà penso che dovrebbe essere integrato direttamente in Discourse…

7 Mi Piace

Quindi ho provato a usare il ramo della soluzione temporanea e, se non si utilizza un set limitato, viene visualizzato il selettore di emoji che rimane in attesa. Inoltre, non permette di chiudere la finestra del selettore. In realtà, rompe anche il selettore di emoji nella finestra di modifica predefinita (lo fa apparire troppo a destra dello schermo e rimane in attesa). Il selettore di emoji con set limitato rimane statico sulla pagina mentre scorri, quindi rimane bloccato dove appare, ma altrimenti funziona. L’ho provato sia su Chrome che su Firefox, entrambi all’ultima versione su macOS.

Ho dato un’occhiata al tuo codice e non ho notato nulla di evidente. Non conosco il framework di Discourse, però. Il mio istinto mi dice che sia legato alle modifiche relative a onShow, ma non faccio molta sviluppo frontend. Potrei dare un’occhiata più approfondita se avrò tempo e, se troverò qualcosa, lo riferirò in un modo o nell’altro.

1 Mi Piace

Sì, ho avuto lo stesso problema. Cliccando sul pulsante React si apre la finestra, ma non viene caricato nulla e, in realtà, sembra bloccare la finestra per me.

3 Mi Piace

Stesso problema di @leighno5, questa estensione blocca le pagine, devo rimuoverla

@angus Ho visto che la tua correzione è stata unita, ma come segnalato sopra, non funziona. C’è la possibilità che tu possa dare un’occhiata?

Ehi ragazzi, solo una nota: James mi ha dato i permessi di push sul repository Retort, quindi farò quanto segue:

  • risolverò completamente questo problema di compatibilità nei prossimi giorni
  • aggiungerò il file .discourse_compatibility qui, in modo che il plugin possa essere agganciato al ramo stable
  • aggiungerò il plugin all’istanza di staging pubblica di Pavilion: try.thepavilion.io
  • aggiornerò mensilmente la compatibilità del plugin (aggancio a stable e compatibilità con tests-passed).

Tenete duro!

13 Mi Piace

Ok, ho aggiornato il plugin per supportare correttamente il nuovo selettore di emoji in Discourse :tada:

È stato testato completamente sia in modalità completa che limitata, e anche su mobile. Puoi provarlo tu stesso qui:

https://try.thepavilion.io/t/heres-something-to-react-to/571

Questo sito viene ricostruito automaticamente ogni 24 ore (aggiornandolo all’ultima versione di Discourse). Quindi, se sei sul ramo tests-passed (il ramo predefinito), è una buona idea testare eventuali plugin personalizzati che stai utilizzando prima di eseguire l’aggiornamento.

Ho anche aggiunto un file .discourse_compatibility al plugin:

Add .discourse_compatibility · gdpelican/retort@35aaad5 · GitHub.

Ciò significa che se stai utilizzando il ramo stable di Discourse, puoi continuare ad aggiornare normalmente e questo plugin rimarrà bloccato su una commit compatibile con l’ultima versione stabile. Per ulteriori informazioni, consulta:

Introducing .discourse-compatibility: pinned plugin/theme versions for older Discourse versions.

In questo caso, l’ultima versione stabile di Discourse (2.5.1) è stata rilasciata due settimane fa, mentre il refactoring del selettore di emoji che ha rotto Retort è stato effettuato 10 giorni fa. Pertanto, la commit stabile per questo plugin è Gobals are being loaded later · gdpelican/retort@92738f0 · GitHub, che ho creato a giugno (testata e funzionante con la versione stabile 2.5.1).

17 Mi Piace

Grazie @angus! Funziona sulla nostra istanza di Discourse!!

3 Mi Piace

@sam Potresti spostare questo di nuovo in #plugin? Grazie.

4 Mi Piace

Grazie mille per averlo fatto!

Una cosa che ho notato (oltre al fatto che funziona, evviva) è che quando clicchi sul pulsante “rispondi”, la pagina si sposta in modo che il selettore di emoji sia in alto (nell’ultima versione di Chrome su macOS); poi, se scorri la pagina, il selettore di emoji si sposta in posizioni strane. Volevo solo segnalare.

3 Mi Piace