Discourse GIFs

Ok danke, eine Frage: Ist die neueste Version des Plugins mit Stable 2.8.1 kompatibel?

1 „Gefällt mir“

Ich bin mir nicht sicher, da wir keine stabile Version betreiben und das Chat-Plugin sich noch im Alpha-Stadium befindet und sich schnell ändert.

3 „Gefällt mir“

Ok, ich hab’s verstanden, vielen Dank.

1 „Gefällt mir“

@RGJ Frage: Wenn der Chat stabil läuft, habe ich dann die neueste Version des Plugins auf meinem Forum?

1 „Gefällt mir“

Nein, das Chat-Plugin befindet sich noch im Alpha-Stadium und ändert sich schnell, und die neueste Version des Chat-Plugins funktioniert nicht auf dem stabilen Zweig von Discourse, daher heften wir es an eine frühere Version.

3 „Gefällt mir“

Oh perfekt, ich hatte die Angelegenheit nicht gut verstanden.

1 „Gefällt mir“

Für diejenigen, die sich für Tenor entscheiden und noch keinen (v1)-Schlüssel haben, funktioniert die Komponente nicht, da sie die erste Version der API verwendet, die veraltet ist. Gelieferte Schlüssel sind jetzt v2 und die Endpunkte haben sich geändert:

2 „Gefällt mir“

Tenor-Unterstützung wurde von @merefield hinzugefügt. Planen Sie, sie für 2.0 zu aktualisieren?

2 „Gefällt mir“

Hallo Falco, ich bin mir nicht sicher, ob du Robert oder mich meinst :sweat_smile: , da es viel mehr Sinn ergibt, Robert. Ich plane es ganz sicher nicht (wahrscheinlich überfordert, es sauber zu machen, aber hey, ich könnte es versuchen!)

2 „Gefällt mir“

Dies ist ein besserer Link, um die Änderungen zu beschreiben: How to migrate from Tenor's V1 API  |  Google for Developers

Ich fürchte, ich bin gerade unter Zeitdruck für ein Projekt, aber ich schaue es mir gerne bald an, wenn es noch offen ist.

3 „Gefällt mir“
FEHLGESCHLAGEN
--------------------
[2022-09-02T18:17:29.927471 #1]  INFO -- : cd /var/www/discourse & su discourse -c 'bundle exec rake themes:update assets:precompile'
ENOENT: no such file or directory, open '/var/www/discourse/plugins/discourse-gifs/plugin.rb'1

Gibt es derzeit bekannte Probleme mit dieser Komponente?

1 „Gefällt mir“

Hallo @Alon1 – es sieht so aus, als ob du versucht hast, diese Theme-Komponente als Plugin in deiner app.yml-Datei zu installieren. Das funktioniert nicht – du solltest die Zeile aus deiner app.yml-Datei entfernen.

Mehr Kontext darüber, warum dies erst jetzt Fehler verursacht, findest du in diesem Beitrag

5 „Gefällt mir“

Ich habe gerade einen PR mit den für Tenor V2 benötigten Änderungen angefordert. Tenor V1-Anmeldeinformationen sind mit V2 kompatibel, daher sollte dies hoffentlich keine Breaking Change sein. Ich habe keine V1-Anmeldeinformationen zum Testen, aber ich habe es mit V2 getestet und es funktioniert für mich hervorragend.

Mein geforkter Branch ist für andere offen, aber bitte benutzen Sie ihn auf eigene Gefahr.

4 „Gefällt mir“

Großartig, danke! Ich habe auf Github geantwortet.

2 „Gefällt mir“

Tatsächlich glaube ich nach dem Testen auch, dass der V1-Schlüssel nicht kompatibel ist und sobald dies zusammengeführt wird, müssen Benutzer ihre Website als Projekt in der Google Cloud Console registrieren und die Tenor-API hinzufügen, einen Schlüssel generieren und ihre Einstellungen aktualisieren:

Aber ich glaube nicht, dass dem geholfen werden kann, und der Wechsel zur neuen API ist eine sehr gute Idee.

2 „Gefällt mir“

Ich werde prüfen, ob ich einen Umschalter für v1/v2 hinzufügen kann, um dies zu verhindern, damit es keine abwärtskompatible Änderung darstellt.

3 „Gefällt mir“

Ob das nicht trotzdem eine Breaking Change im Upstream sein wird, wenn die ursprüngliche API außer Betrieb genommen wird? Ich würde das Team um eine Einschätzung bitten, ob das notwendig ist. Es würde den Leuten jedoch mehr Zeit geben. Lassen Sie uns das im PR besprechen.

2 „Gefällt mir“

Danke für den PR @tnoor-co!

Wenn ich darüber nachdenke, da Tenor V1 API-Schlüssel jetzt unmöglich zu bekommen sind, wird selbst die Wartung dieses Theme-Komponenten-Teils von Tenor V1 zu einer Belastung, da ein Entwickler einen funktionierenden API-Schlüssel benötigt, um ihn lokal zu testen.

Vor diesem Hintergrund würde ich sagen, wir entfernen Tenor V1 jetzt aus der Komponente. Administratoren, die Tenor wie @PaulinaMX verwenden, können mit dem Update warten und einen V2 API-Schlüssel erhalten, bevor sie aktualisieren. Wir können auch eine tenor-v1-Branch eine Weile für Leute aufbewahren, die sie benötigen.

Was denkst du @merefield?

5 „Gefällt mir“

Einverstanden.

Unter Berücksichtigung, dass @tnoor-co hier enorm viel Aufwand betrieben hat (und ich fürchte, ihn um mehr zu bitten), sagt mir mein Gefühl, dass wir wahrscheinlich aufs Ganze gehen und den V1-Code vollständig entfernen sollten, um die Einfachheit und Klarheit der Einstellungen und des Codes im Hauptzweig zu wahren.

Es fühlt sich einfach nach viel Komplexität an, um das Unvermeidliche zu vermeiden :slight_smile:

4 „Gefällt mir“

Klingt gut für mich. Ich werde diese Woche ein Update veröffentlichen, um v1 vollständig zu entfernen.

5 „Gefällt mir“