PatchworkBoy / homebridge-edomoticz

Domoticz Homebridge-Plugin
Other
118 stars 44 forks source link

Thermostat are not visible any more with Domoticz 2024.x #279

Closed MadPatrick closed 3 months ago

MadPatrick commented 4 months ago

Hi, I'm trying out Homebridge with edomoticz, but hte themrostats are not visible in Accesoiries. When i try on a VM to use it with Domotivz 2023.2 they are visible

DewGew commented 4 months ago

For latest domoticz 'Type': 'Thermostat' is renamed to 'Type': 'Setpoint'. maybe this can be the issue? https://github.com/PatchworkBoy/homebridge-edomoticz/blob/c3bb69422b7288d5a5a829ab49d029ebfc6992f0/lib/domoticz_accessory.js#L636

MadPatrick commented 4 months ago

I've tried your suggestion, but did not fix the problem

DewGew commented 4 months ago

Checkout this issue #274 maybe you find somthing there to fix this

MadPatrick commented 4 months ago

That worked. Thanks !!

Probably wants || this.Type == "Setpoint" adding at line 2226…

homebridge-edomoticz/lib/domoticz_accessory.js

Line 2226 in c3bb694

case this.Type == "Heating" || this.Type == "Thermostat":

PatchworkBoy commented 3 months ago

Pull reqs merged and a few additional tweaks. Closing. https://github.com/PatchworkBoy/homebridge-edomoticz/commit/0d52ac714bdaa59919ecc9126feadae0b0daad70