Unser „Reaktions“-Button funktioniert nach dem neuesten Update nicht.
Beim Klicken auf das Emoji-D-Icon finde ich keine Fehler in der Konsole. Mit meinem begrenzten Wissen darüber, wie es implementiert ist, kann ich den Fehler nicht finden. Hoffentlich kann jemand helfen … (Ich habe versucht, das Repository durchzusehen, aber nichts gefunden – bei meinem begrenzten Wissen über die Erstellung von Plugins für Discourse (!))
Hat jemand herausgefunden, was das Problem sein könnte? Im Codebase des Plugins hat sich schon eine Weile nichts geändert, also vermute ich, dass sich etwas im Discourse-Codebase geändert hat, was dies verursacht (ich habe jedoch keine offensichtlichen Änderungen gesehen, die meiner Meinung nach einen Einfluss darauf hätten…).
Es gibt mehrere Updates von Discourse, die in diesem Plugin behandelt werden müssen.
Ich habe einen PR erstellt, der als vorübergehende Lösung dienen kann. Ich werde wahrscheinlich für die Art und Weise, wie die Lösung funktioniert, in die feurigen Höhlen des Tartarus verdammt werden, aber es könnte als Überbrückung dienen @gdpelican
@Jite Du scheinst interessiert zu sein. Möchtest du ein bisschen mehr über die Discourse-Entwicklung und die Funktionsweise dieses Plugins lernen, um es zu pflegen? Ich kann dir helfen, die Feinheiten der Discourse-Codebasis zu verstehen.
Ich hätte das Angebot gerne angenommen, aber aktuell habe ich einfach zu viel um die Ohren. Ich habe bereits ein paar kleinere Plugins erstellt, aber meine Ruby-Erfahrung ist noch so gering, dass ich mich bisher nicht tief in die Interna einarbeiten konnte, hehe.
Ich bleibe aber dabei, also falls ich mehr Zeit dafür finde, melde ich mich gerne bei dir!
Ich werde mir die Korrektur auch mal ansehen, denn die Instanz, auf der ich das Plugin nutze, scheint es wirklich zu mögen
Außerdem frage ich mich wirklich, wie ich diese Zusammenführung in den Logs nicht gefunden habe, ich habe sie gestern doch wie ein Buch gelesen! Das war schlecht von mir ;D
Ich habe also deinen Branch mit der vorübergehenden Lösung ausprobiert. Wenn du keinen begrenzten Satz verwendest, wird der Emoji-Picker angezeigt und spinnt. Außerdem lässt er sich nicht schließen. Tatsächlich wird dadurch auch der Emoji-Picker im Standard-Editor-Fenster beschädigt (erscheint weit rechts auf dem Bildschirm und spinnt). Der Emoji-Picker für den begrenzten Satz bleibt beim Scrollen statisch auf der Seite stehen, also bleibt er dort, wo er erscheint, aber ansonsten funktioniert er. Ich habe es sowohl mit der neuesten Version von Chrome als auch von Firefox unter OSX getestet.
Ich habe kurz in deinen Code geschaut, aber nichts Offensichtliches gefunden. Ich kenne das Discourse-Framework jedoch nicht. Mein Bauchgefühl sagt mir, dass es mit den Änderungen im Zusammenhang mit onShow zu tun hat, aber ich mache nicht viel Frontend-Entwicklung. Falls ich Zeit habe, werde ich vielleicht genauer hinsehen, und falls ich etwas finde, werde ich es auf jeden Fall melden.
Ja, ich hatte genau das gleiche Problem. Beim Klicken auf den React-Button öffnet sich das Fenster, aber es lädt nichts, und für mich friert das Fenster tatsächlich ein.
Diese Seite wird alle 24 Stunden automatisch neu aufgebaut (und dabei auf die neueste Discourse-Version aktualisiert). Wenn du also auf tests-passed (dem Standardbranch) bist, ist es ratsam, vor einem Update alle benutzerdefinierten Plugins, die du verwendest, dort zu testen.
Außerdem habe ich eine .discourse_compatibility-Datei zum Plugin hinzugefügt:
Das bedeutet, dass du, wenn du den stable-Branch von Discourse verwendest, ganz normal weiter aktualisieren kannst, während dieses Plugin auf einem Commit festgelegt bleibt, der mit der neuesten stabilen Version funktioniert. Weitere Informationen findest du hier:
Eine Sache, die mir aufgefallen ist (abgesehen davon, dass es funktioniert, ja): Wenn du auf den „Antworten“-Button klickst, springt die Seite so, dass der Emoji-Auswahlbereich oben erscheint (aktuellster Chrome, macOS). Wenn du dann auf der Seite scrollst, verschiebt sich der Emoji-Auswahlbereich an seltsame Stellen. Ich wollte das nur melden.