search

fdegier / homebridge-jablotron-alarm

Homebridge plugin for connecting Jablotron JA-100 alarm systems to Homekit.
The Unlicense
26 stars 6 forks source link

Add thermometers #64

Closed dfridrich closed 2 years ago

dfridrich commented 2 years ago

It would be great if this Homebridge extension would also show temperature senzors (they are accessible from jablonet.net).

image

fdegier commented 2 years ago

That would be awesome indeed, I do not have any temperature sensors unfortunately. Would you be able to help?

misncz commented 2 years ago

Are you using wired (to bus) or wireless sensors?

dfridrich commented 2 years ago

Are you using wired (to bus) or wireless sensors?

I am using all wired.

misncz commented 2 years ago

Are you using wired (to bus) or wireless sensors?

I am using all wired.

Ok. My friend can loan me one so I give it a try and will see what we can do to support them

dfridrich commented 2 years ago

Are you using wired (to bus) or wireless sensors?

I am using all wired.

Ok. My friend can loan me one so I give it a try and will see what we can do to support them

If you'll need any help, don't hesitate to contact me as well. Good luck ;-)

misncz commented 2 years ago

@dfridrich I implemented support for thermometers (not thermostats as I don't have any) and raised a pull request. Enjoy once it gets merged and released by @fdegier

misncz commented 2 years ago

@dfridrich you need to run config helper to receive thermometers config to be added to homebridge

fdegier commented 2 years ago

Released in v2.4.0

dfridrich commented 2 years ago

@misncz thanks for great job! Is there possibility to add thermostats too? It would be very useful.

misncz commented 2 years ago

@dfridrich unfortunately I don't have any thermostats available to be able to do that

misncz commented 2 years ago

@dfridrich looks like I'll be able to get one thermostat for testing the week after next one. I let you know if I'm able to progress then

dfridrich commented 2 years ago

@misncz thanks, if you would to have some more thermostats for test, I could provide some.

dfridrich commented 2 years ago

@misncz did you have time to think about thermostats implementation? May I help somehow?

misncz commented 2 years ago

@dfridrich yes I did. I'm able to read current temperature only (not temperature set on thermostat). The problem is that to be able to read/control more thermostat options we'd need to upgrade Jablotron API version which is not an easy task. With currently implemented API version the thermostat functionality is limited to thermometer which is not what you'd like to have I assume

CC: @fdegier