Salut tous le monde,
Je crois qu’on est plusieurs à avoir une instance Nextcloud donc j’ai souhaité nous ajouter l’intégration de l’application Talk. L’idée est d’avoir une alternative à Telegram pour communiquer avec votre instance Gladys.
En attendant d’avoir l’intégration dans Gladys, si vous voulez voir la PR en cours, elle est visible ici [WIP] Add Nextcloud Talk service by bertrandda · Pull Request #1388 · GladysAssistant/Gladys · GitHub
Pour ceux qui veulent tester, une image docker de test est disponible bertrandda/gladys:nextcloud-talk
Si vous avez besoin de la doc pour paramétrer la connexion à votre instance Nextcloud c’est ici
N’hésitez pas si vous avez des retours
Je vais faire une installation pour tester, je te ferai un feedback.
Merci @VonOx ! Tenez moi au courant des tests et je ferais ensuite une review technique dès que j’ai un peu de temps 
Good job @bertrandda
Si je comprends c’est monodirectionnel ? Talk vers Gladys ?
Je demande ça car l’envoi de message depuis une scène ne fonctionne pas de mon côté
Merci d’avoir pris le temps de tester 
J’ai peut être manqué la partie scène, je vais investiguer. L’objectif est bien d’avoir l’ensemble des fonctions de communications (dans les 2 sens) disponible via Nextcloud Talk
Ok tiens miens au jus
J’avais rien de spécial dans les logs dans tous les cas
C’était bien une erreur de ma part, c’est fixé et l’image est à jour
OK je retest ce soir
Edit: Test concluant, c’est tout bon de mon côté @bertrandda
@bertrandda Regarding this PR, I don’t know if you had seen my last review:
I hadn’t noticed that, I’ll go over it again.
I rewrote the service following the reviews and improved its behavior after saving a new configuration. A new image is available bertrandda/gladys:nextcloud-talk
@bertrandda Thanks for the changes, that’s fine with me like this 
I just found a small typo but nothing serious 
Could you rebase onto master to check that it works well on Node.js 18 and fix the typo, and then if everything’s good I’m ready to merge it into master!
Thanks for the work done, the PR is really clean
All set, everything’s up to date, and thanks to switching to Node 18 we can now use the native AbortController instead of the polyfill 
Great idea! Thanks for the feature. 
We could do the same with the Nextcloud Calendar service, right?! 
You want to synchronize your calendar with Gladys, is that right? It’s already possible with the CalDAV integration, you can check here CalDAV | Gladys Assistant
Yes I had seen @bertrandda, thanks.
I misspoke, sorry. I meant to use these 2 Nextcloud features in gladys rather than having 3 different apps.
[
I had thought about it but in the end they are 2 different accounts that you configure (the user’s for the calendar and the bot’s for Nextcloud Talk) so unifying the configurations in this case isn’t feasible. However, if other Nextcloud apps become compatible with Gladys it would indeed be interesting to have a single place to configure the user account for all integrations of an NC instance.
Thanks — I’ll take care of it tonight or tomorrow. Since the modifications to make are only at the level of the test files and the rest of the integration shouldn’t change, for those who already want to test the integration an image is available bertrandda/gladys:nextcloud-talk
Quick bump, I just updated the test image if anyone’s interested
