Retours Discourse

Notre bouton de « réaction » ne fonctionne plus après la dernière mise à jour.

Je ne trouve aucune erreur dans la console lorsque je clique sur l’icône d’emoji. Avec mes connaissances limitées sur son implémentation, je n’arrive pas à repérer le bug. J’espère que quelqu’un y arrivera :smiley: … (J’ai essayé d’examiner le dépôt, mais sans succès - compte tenu de mes connaissances limitées sur la création de plugins pour Discourse (!))

3 « J'aime »

Même problème. Je ne peux pas cliquer sur mon site web.

2 « J'aime »

Je peux confirmer que ce bogue existe bel et bien. Rien ne se produit après avoir cliqué dessus.

2 « J'aime »

Je peux aussi confirmer. Lorsque je clique sur l’icône des réactions pour la définir, rien ne s’affiche, il ne se passe rien…

2 « J'aime »

Ajout de #plugin:broken-plugin pour le moment. Faites-le-moi savoir une fois corrigé.

2 « J'aime »

Quelqu’un a-t-il trouvé ce qui pourrait être le problème ? Rien n’a changé dans le code du plugin depuis un certain temps, donc je suppose que quelque chose a changé dans le code de Discourse et provoque cela (je n’ai pas vu de changements évidents qui auraient un impact, cependant…).

1 « J'aime »

Il y a plusieurs mises à jour de Discourse qui doivent être prises en compte dans ce plugin.

J’ai créé une PR qui peut servir de solution temporaire. Je serai probablement condamné aux enfers de Tartare pour la façon dont fonctionne cette correction, mais cela pourrait servir d’arrangement provisoire @gdpelican

@Jite, tu sembles motivé. Aimerais-tu en apprendre un peu plus sur le développement de Discourse et le fonctionnement de ce plugin pour aider à sa maintenance ? Je peux t’aider à comprendre les subtilités de la base de code de Discourse.

8 « J'aime »

J’aurais adoré accepter votre offre, mais pour le moment, j’ai déjà beaucoup trop de choses à faire. J’ai déjà créé quelques petits plugins, mais mon expérience avec Ruby est si limitée que je n’ai pas encore pu plonger profondément dans les mécanismes internes, hehe.
Je resterai cependant dans les parages, donc si je trouve plus de temps pour cela, je vous le ferai savoir avec plaisir ! :slight_smile:

Je jetterai aussi un coup d’œil à la correction, car l’instance où j’utilise le plugin semble vraiment l’apprécier :smiley:

4 « J'aime »

D’ailleurs, je me demande vraiment comment je n’ai pas trouvé cette fusion dans les journaux, je les ai lus comme un livre hier ! C’était de ma part une grosse erreur ;D

3 « J'aime »

Fais-moi signe :slight_smile: Je serais ravi de t’aider à prendre tes marques avec Discourse.

5 « J'aime »

:sob:
J’adore ce plugin, j’aimerais qu’il soit maintenu. En fait, je pense qu’il devrait être intégré directement dans Discourse…

7 « J'aime »

J’ai donc essayé d’utiliser votre branche de correctif temporaire et, si vous n’utilisez pas un ensemble limité, le sélecteur d’emojis s’affiche et tourne indéfiniment. De plus, il n’est pas possible de fermer la fenêtre du sélecteur. Cela affecte également le sélecteur d’emojis dans la fenêtre d’édition par défaut (il apparaît tout à droite de l’écran et tourne). Le sélecteur d’emojis avec un ensemble limité reste statique sur la page lors du défilement, donc il reste bloqué là où il apparaît, mais sinon cela fonctionne. J’ai testé cela sur les dernières versions de Chrome et Firefox sous macOS.

J’ai jeté un coup d’œil à votre code, mais je n’ai rien trouvé d’évident. Je ne connais pas le framework Discourse, cependant. Mon intuition me dit que cela pourrait être lié aux modifications concernant onShow, mais je ne fais pas beaucoup de développement frontend. Je pourrai peut-être approfondir l’analyse si j’ai le temps, et si je trouve quelque chose, je le signalerai d’une manière ou d’une autre.

1 « J'aime »

Oui, j’ai eu exactement le même problème. Cliquer sur le bouton réagir ouvre la fenêtre, mais rien ne se charge, et cela semble même figer la fenêtre pour moi.

3 « J'aime »

Même problème que @leighno5, cette extension fige les pages, je dois la désinstaller

@angus J’ai vu que ta correction a été intégrée, mais comme signalé plus haut, elle ne fonctionne pas. As-tu la possibilité d’y jeter un coup d’œil ?

Salut à tous, juste une petite note pour vous dire que James m’a donné les permissions de push sur le dépôt Retort. Voici donc ce que je vais faire :

  • résoudre complètement ce problème de compatibilité dans les prochains jours
  • ajouter le fichier .discourse_compatibility ici afin que le plugin puisse être verrouillé sur la branche stable
  • ajouter le plugin à l’instance de préproduction publique de Pavilion : try.thepavilion.io
  • mettre à jour mensuellement la compatibilité du plugin (verrouillage sur stable et compatibilité avec tests-passed).

Tenez bon !

13 « J'aime »

Ok, j’ai mis à jour le plugin pour prendre correctement en charge le nouveau sélecteur d’emojis dans Discourse :tada:

Il a été testé en mode complet et limité, ainsi que sur mobile. Vous pouvez le tester vous-même ici :

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

Ce site est reconstruit automatiquement toutes les 24 heures (ce qui le met à jour avec la dernière version de Discourse). Donc, si vous êtes sur tests-passed (la branche par défaut), il est conseillé de tester tous les plugins personnalisés que vous utilisez avant de mettre à jour.

J’ai également ajouté un fichier .discourse_compatibility au plugin :

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

Cela signifie que si vous exécutez la branche stable de Discourse, vous pouvez continuer à mettre à jour normalement et ce plugin restera figé sur un commit compatible avec la dernière version stable. Pour en savoir plus, consultez :

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

Dans ce cas, la dernière version stable de Discourse (2.5.1) datait de deux semaines, et le refactoring du sélecteur d’emojis qui a cassé Retort remonte à 10 jours. Le commit stable pour ce plugin est donc Gobals are being loaded later · gdpelican/retort@92738f0 · GitHub, que j’ai créé en juin (testé et fonctionnel avec la version stable 2.5.1).

17 « J'aime »

Merci @angus ! Cela fonctionne sur notre instance Discourse !!

3 « J'aime »

@sam Pourrais-tu déplacer cela dans #plugin ? Merci.

4 « J'aime »

Merci beaucoup pour cela !

Une chose que j’ai remarquée (en plus du fait que ça fonctionne, super) : lorsque vous cliquez sur le bouton de réponse, la page se redresse pour que le sélecteur d’émojis soit en haut (dernière version de Chrome, macOS). Ensuite, si vous faites défiler la page, le sélecteur d’émojis se déplace vers des endroits étranges. Je voulais simplement signaler ce problème.

3 « J'aime »