dehsgr / node-red-contrib-evohome

This provides nodes for controlling Honeywell Evohome
ISC License
5 stars 7 forks source link

Reading Hot Water #4

Closed benneaves closed 2 years ago

benneaves commented 5 years ago

At the moment only heating zones are reported, hot water values are not broadcast On other EvoHome API decoders i have seen, hotwater appears in an unnamed zone. Seems to work ok in this code https://gist.github.com/davesarg/a453915074c5508dda8a

dehsgr commented 5 years ago

As I don't have any Evohome myself I'm currently not able to implement this. Feel free to (find one to) contribute.

benneaves commented 5 years ago

looks like the code to implement is already there, ill make some tweeks to see if it works

if(locations[0].devices[deviceId].zoneID == thermostats[thermoId].zoneId) { if(locations[0].devices[deviceId].name == "") { // Device name is empty // Probably Hot Water // Do not store console.log("Found blank device name, probably stored hot water. Ignoring device for now."); } else { var msgout = { payload : { id: thermostats[thermoId].zoneId, name : locations[0].devices[deviceId].name.toLowerCase(), currentTemperature : thermostats[thermoId].temperatureStatus.temperature, targetTemperature : thermostats[thermoId].setpointStatus.targetHeatTemperature }

davies8172 commented 4 years ago

Did anyone get anywhere with this fix? I would love to bring in hot water temperature into my dashboard if available?

tstaddon commented 3 years ago

I did find some stuff on the Automated Home forum but don't have HW to test the code:

Automatedhome.co.uk

@dehsgr - I don't know if it's relevant but some of the authentication code in andremain's SmartThings project has been updated very recently and could be worth a collaboration:

github.com/andremain/EvohomeSmartthings2020

dehsgr commented 3 years ago

@tstaddon thanks for this hint. I‘ll take a look when there is some time. Currently I‘m busy and I‘ve no Evohome for testing.

dehsgr commented 2 years ago

No time for implementation and no platform for testing those changes here. Feel free to contribute. I'll close this now.