Nextcloud Talk Integration

Development
1

Hello everyone,

I think several of us have a Nextcloud instance, so I wanted to add the integration of the Talk application for us. The idea is to have an alternative to Telegram to communicate with your Gladys instance.

While waiting for the integration in Gladys, if you want to see the ongoing PR, it is available here https://github.com/GladysAssistant/Gladys/pull/1388

2

For those who want to test, a test docker image is available bertrandda/gladys:nextcloud-talk

If you need the documentation to configure the connection to your Nextcloud instance, it’s here

Do not hesitate if you have feedback

3

I will do an installation to test, I will give you feedback.

4

Thanks @VonOx! Keep me updated on the tests and I’ll do a technical review as soon as I have some time :slight_smile:

5

Good job @bertrandda

image
image701×450 26.2 KB

If I understand correctly, it’s one-way? Talk to Gladys?
I’m asking because sending messages from a scene doesn’t work on my end.

6

Thank you for taking the time to test :pray:

I might have missed the scene part, I’ll investigate. The goal is indeed to have all communication functions (both ways) available via Nextcloud Talk

7

Got it, I’ll keep you posted

In any case, there was nothing special in the logs

8

It was indeed my mistake, it’s fixed and the image is up to date

9

OK, I’ll retest tonight

Edit: Test successful, all good on my end @bertrandda

10

@bertrandda Regarding this PR, I don’t know if you had seen my last review:

11

I hadn’t noticed that, I’ll go over it again.

12

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

13

@bertrandda Thanks for the changes, that’s fine with me like this :+1:

I just found a small typo but nothing serious :smile:

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

14

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 :ok_hand:

15

Great idea! Thanks for the feature. :slight_smile:
We could do the same with the Nextcloud Calendar service, right?! :thinking:

16

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

17

Yes I had seen @bertrandda, thanks. :slight_smile:
I misspoke, sorry. I meant to use these 2 Nextcloud features in gladys rather than having 3 different apps. :slight_smile:

18

[

19

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

20

Quick bump, I just updated the test image if anyone’s interested