SergiuToporjinschi / node-red-contrib-heater-controller

Heater controller for node-red dashboard
GNU General Public License v3.0
18 stars 17 forks source link

Add checkbox 'If msg arrives on input, pass through to output:' #12

Closed HanSolo72 closed 5 years ago

HanSolo72 commented 5 years ago

First, thanks to your great work!

Til now I used the default slider. There is a checkbox If msg arrives on input, pass through to output: that avoids endless loops. Please add the same option, because I just created an endless loop.

SergiuToporjinschi commented 5 years ago

Hi, Can you do a fork? I'm not sure what you mean. I will review the changes and I can say if I can merge it.

HanSolo72 commented 5 years ago

Hi, I can't, because I'm not a experienced coder. It's just an enhancement request. I uploaded a screenshot: https://i.imgur.com/jl9IDdN.png

My problem is: When the target temperature is set from Alexa, I want to pass the value by topic userTargetValue to the heater node and the heater note sets the target value. That is my endless loop.

SergiuToporjinschi commented 5 years ago

I don't have time now, but I will take this in consideration on next release thank you for your idea

SergiuToporjinschi commented 5 years ago

@HanSolo72 is already outputting the inserted values but is outputting the entire set.

{"currentTemp":23,"targetValue":22,"isUserCustom":true,"currentSchedule":{"temp":22,"day":"Wednesday","time":"16:40"},"nextSchedule":{"temp":19,"day":"Wednesday","time":"23:59"},"currentHeaterStatus":"off","userTargetValue":22}

check also the schema from the bottom of the readme file.