search

vanackej / risco-mqtt-local

Provide Risco alarm system integration to Home assistant using local TCP communication (no cloud required) and MQTT
MIT License
24 stars 11 forks source link

Outputs development #48

Open markxroberts opened 2 years ago

markxroberts commented 2 years ago

I think these are the changes I introduced for outputs.

markxroberts commented 2 years ago

I've also amended the zone status so this updates

markxroberts commented 2 years ago

I think I've now managed to add these commits separately. I'm happy to test Outputs once you've had a look. I've not retested this since adding these commits, but think it reflects all the changes. The status update for outputs is a workaround as previously described. I've reformatted the REGEX as per your changes to topics.

658c64e I've added some additional code for the bypass and battery attributes. As I read the code, this info is only published to HA at discovery, and so I think we need to re-update the attributes specifically on ZStatusChanged (this was how I worked it when I set it up).

9b26be2 is self-explanatory. Otherwise, this option doesn't do anything in the HA front end!

9e65127 publishes HA autodiscovery only once instead of twice.

markxroberts commented 2 years ago

Latest commit should resolve the issue where state within HA not defined at first install. Issue of retained states should not now be a problem as HA online status monitored