Set device as HeaterCooler instead of Thermostat. - #5
Conversation
|
Updated for the new mymodlet.com in 5453fd7. |
|
I got it working after installing https://github.com/cheeriojs/cheerio. |
caf3ee7 to
0eb9482
Compare
|
It turns out the new website allows controlling the A/C when the thermostat is offline! I've updated this pull request's error checking in 9427bf0 to only error out with "No response" if the modlet itself is offline, and otherwise show "0°" for the current temperature to indicate if the thermostat is broken.
|
|
Thanks for this. So far your branch is the only way I've found to get the modlet working with homebridge again after the web update. |
picnicbob
left a comment
picnicbob
left a comment
There was a problem hiding this comment.
I'm not sure how to do this myself or I would. Not a frequent user of GitHub.com
| return callback(new Error('Modlet broken.')); | ||
| } | ||
|
|
||
| this.api.log(this.name, 'set heating / cooling active: ' + !this.powerOn); |
There was a problem hiding this comment.
setActiveState() is not necessarily always called to toggle, it can be called with the existing value. Use value instead of !this.powerOn
|
|
||
| this.api.log(this.name, 'set heating / cooling active: ' + !this.powerOn); | ||
|
|
||
| this.powerOn = !this.powerOn; |
There was a problem hiding this comment.
(Use value instead of !this.powerOn here too obviously)
d4007b2 to
54565fa
Compare
54565fa to
cf3e48f
Compare
|
Oh, good catch @picnicbob! Fixed in cf3e48f. |
|
just wanna leave this command here if anyone is trying to install this branch
installs the plugin @DFurnes branch commit |
|
Apologies for the long absence -- I moved and don't have a AC unit that can use the SmartAC devices anymore. I'll merge this in and do a new release. If anyone would like to take over stewardship of the project, please reach out. |
5 participants
Thanks for the fantastic plugin!!
I wanted to be able to tap the accessory in the Home app in order to turn on & off my air conditioner. Switching the service to be a
HeaterCoolerseems to have done the trick.After swapping that, I was also able to toggle the "state" from
COOLINGtoIDLEbased on whether the thermometer read lower than the target, so that the app would show the correct icon and verb: