PJCzx / homebridge-thermostat

Supports thermostat devices on HomeBridge Platform
Apache License 2.0
34 stars 71 forks source link

heatOnly option #50

Closed G0sch closed 4 years ago

G0sch commented 4 years ago

Hey

Should the "heatOnly" Option remove the "COOL" option in Home App? If yes it looks like its not working.

image

![image](https://user-images.githubusercontent.com/51472553/73339033-1c81d180-4278-11ea-8efd-135b38baee39.png)
PJCzx commented 4 years ago

@clvz do you have any input for @G0sch ?I'm not using my own plugin anymore ^^

clvz commented 4 years ago

Hello @G0sch ,

This is strange. Here's my config, that works. It doesn't seem much different from yours:

{
       "accessory": "Thermostat",
       "name": "Thermostat Séjour",
       "apiroute": "http://127.0.0.1:5000/room/sejour",
       "currentHumidity": false,
       "heatOnly": true,
       "maxTemp": 22.0,
       "minTemp": 7.0,
       "model": "ThermoPy",
       "serial": "0123",
    }

On your side, have you tried:

  1. restarting homebridge
  2. killing / reopening the Homekit app on your iDevice so that the config update is taken into account?
G0sch commented 4 years ago

Hello @clvz

Yes tried it several time. Even tried to re-install the plugin :-)

thx g0sch

G0sch commented 4 years ago

@clvz I cloned the repo and added "validValues: [0,1]" - looks like it is working now :-)

PJCzx commented 4 years ago

@G0sch Great, can you do a pull request ? this will allow to provide your work to other people that could be interesed ;) Further more, I'll publish it on NPM for an easy update

G0sch commented 4 years ago

sorry for the late response - i'm not so familiar with GitHub :-/ Have to read how it works in detail

PJCzx commented 4 years ago

it's quite easy, You make a fork, then you make your change. You will be able to make a pull request from your repo to mine ;)

G0sch commented 4 years ago

done - could you check it, please :-)

PJCzx commented 4 years ago

Corrected in #51 by @G0sch thx !

PJCzx commented 4 years ago

Published on npm as homebridge-thermostat@0.0.52