Nextcloud Talk Integration

Hallo zusammen,

Ich glaube, wir sind mehrere, die eine Nextcloud-Instanz haben, also wollte ich uns die Integration der Talk-App hinzufügen. Die Idee ist, eine Alternative zu Telegram zu haben, um mit eurer Gladys-Instanz zu kommunizieren.

Bis die Integration in Gladys verfügbar ist, könnt ihr euch den laufenden PR hier ansehen: https://github.com/GladysAssistant/Gladys/pull/1388

Für diejenigen, die testen möchten, ist ein Docker-Testimage verfügbar bertrandda/gladys:nextcloud-talk

Falls Sie die Dokumentation zur Einrichtung der Verbindung zu Ihrer Nextcloud-Instanz benötigen, finden Sie diese hier

Zögern Sie nicht, Feedback zu geben

Ich werde eine Installation zum Testen machen, ich gebe dir Feedback.

Danke @VonOx! Haltet mich über die Tests auf dem Laufenden und ich werde dann eine technische Überprüfung machen, sobald ich etwas Zeit habe :slight_smile:

Gute Arbeit @bertrandda

Verstehe ich das richtig, dass es nur in eine Richtung geht? Also Talk zu Gladys?
Ich frage, weil das Senden von Nachrichten von einer Szene aus bei mir nicht funktioniert.

Danke, dass du dir die Zeit genommen hast, es zu testen :pray:

Vielleicht habe ich den Szenen-Teil verpasst, ich werde das untersuchen. Das Ziel ist es, alle Kommunikationsfunktionen (in beide Richtungen) über Nextcloud Talk verfügbar zu machen.

Okay, ich halte dich auf dem Laufenden

In jedem Fall gab es nichts Besonderes in den Logs

Es war tatsächlich mein Fehler, es ist behoben und das Bild ist aktuell

OK, ich teste heute Abend nochmal

Edit: Test erfolgreich, alles gut bei mir @bertrandda

@bertrandda Bezüglich dieses PRs, ich weiß nicht, ob du meine letzte Review gesehen hast:

Das hatte ich tatsächlich nicht gesehen, ich werde nochmal drüber schauen

Ich habe den Dienst nach den Reviews überarbeitet und die Funktionsweise nach dem Speichern einer neuen Konfiguration verbessert. Ein neues Image ist verfügbar bertrandda/gladys:nextcloud-talk

@bertrandda Danke für die Änderungen, so ist es für mich in Ordnung :+1:

Ich habe nur einen kleinen Tippfehler gefunden, aber nichts Ernstes :smile:

Ich würde gerne, dass du das in Bezug auf master rebase, um zu prüfen, ob es mit Node.js 18 funktioniert + den Tippfehler korrigierst, und wenn alles klappt, bin ich bereit, es in master zu mergen!

Danke für die Arbeit, die PR ist wirklich sauber

Alles ist jetzt auf dem neuesten Stand, und dank des Upgrades auf Node 18 können wir jetzt den nativen AbortController statt des Polyfills verwenden :ok_hand:

Super Idee! Danke für die Funktion. :slight_smile:
Könnten wir das nicht auch mit dem Kalenderservice von Nextcloud machen, oder?! :thinking:

Möchtest du deinen Kalender mit Gladys synchronisieren? Das ist bereits mit der CalDAV-Integration möglich. Du kannst hier nachschauen: CalDAV dans Gladys : synchroniser iCloud, Google Agenda, Synology et Nextcloud | Gladys Assistant

Ja, ich hatte es gesehen @bertrandda, danke. :slight_smile:
Ich habe mich falsch ausgedrückt, entschuldige. Ich wollte sagen, dass ich diese beiden Nextcloud-Funktionen in Gladys nutzen möchte, anstatt drei verschiedene Apps zu haben. :slight_smile:

Toll :sunglasses:

Ich habe eine Review gemacht, ich habe nur 2 Mini-Kommentare zu den Tests, aber das ist wirklich nur Kleinkram.

Meine Review: https://github.com/GladysAssistant/Gladys/pull/1388#pullrequestreview-1220274969

Ich würde aber gerne noch etwas mehr Testen mit der finalen Version dieses PRs haben, kannst du bitte einen letzten Docker-Build machen?

Falls danach jemand einen Test machen könnte, um sicherzustellen, dass alles funktioniert (da sich seit dem letzten Test einiges geändert hat: Node 18, etc…)

Danach ist für mich alles bereit, um das in die Produktion zu bringen!

Daran hatte ich auch gedacht, aber am Ende sind es zwei verschiedene Konten, die konfiguriert werden (das des Benutzers für den Kalender und das des Bots für Nextcloud Talk). Daher ist es in diesem Fall nicht möglich, die Konfigurationen zu vereinheitlichen. Allerdings wäre es interessant, wenn andere Nextcloud-Apps mit Gladys kompatibel werden, einen einzigen Ort zu haben, an dem das Benutzerkonto für alle Integrationen einer NC-Instanz konfiguriert werden kann.

Danke, ich kümmere mich heute Abend oder morgen darum. Da die Änderungen nur auf Ebene der Testdateien liegen und der Rest der Integration voraussichtlich unverändert bleibt, ist für alle, die die Integration bereits testen möchten, ein Bild verfügbar bertrandda/gladys:nextcloud-talk

Kleines Update, ich habe gerade das Testbild aktualisiert, falls jemand interessiert ist