search

guerrerotook / securitas-direct-new-api

This repository contains the new securitas direct API that can be integrated in Home Assistant
Apache License 2.0
76 stars 30 forks source link

Sentinel sensors in PT #138

Closed pmos69 closed 11 months ago

pmos69 commented 1 year ago

Hi,

In Portugal, the API returns "SENTINEL COMFORTO" as the description of the service. I did a quick workarround to get it working by changing: SENTINE_CONFORT = "SENTINEL CONFORT" to SENTINE_CONFORT = "SENTINEL COMFORTO" in init.py

...but ithis is just a dirty workarround. The country should be checked and the apropriate string used.

I don't know what the apropriate place to do this check would be, so I didn't try to do it correctly and do a pull request.

guerrerotook commented 1 year ago

That is a very good finding, sorry, my installation is in Spanish that is where I'm testing the code. I will take a look at that.

Borty97 commented 1 year ago

Maybe it would be great to select the country and then define variables according to this? It's easy to do?

guerrerotook commented 1 year ago

I already have some configuration that is dependan of the language, like the URL, so I can add this to the settings.

Tanjohnson99 commented 1 year ago

Here in Portugal i don't have the sentinel in HA. Only 1 entity with state of the alarm. Captura de ecrã 2023-08-17 145636

drk1ss commented 1 year ago

@Tanjohnson99 make changes from the first post in /custom_components/securitas/init.py file, reboot. that helped to me

Tanjohnson99 commented 1 year ago

Right! Working . Thanks.