search

mattdavis90 / node-red-contrib-tado-client

Tado web API client node for Node Red
MIT License
23 stars 16 forks source link

Enbled AC by adding Multioverlays #44

Open kneip68 opened 2 years ago

kneip68 commented 2 years ago

Definitely not perfect yet, but at least it works with my AC controls. Created 2 new API Calls for MultiOverlays (both are ending on "NEW" in the englisch translation) for setOverlays and clearOverlays, but left the old calls in. For that I referenced the master branch of node-tado-client in the dependencies in package.json. Not sure

Also extended the GUI with some parameters (Swing, Light) but currently they are still hard coded in the API call, as I was not yet able to figure out what needs to be done to pass them on. Will attach a flow where I use the functions in issue #40 .

mattdavis90 commented 2 years ago

Looks good. Thanks for the work. When I was doing the multi-overlay stuff before I was trying to integrate it into the UI. I was using the following sources to try to get it working

https://github.com/node-red/node-red/blob/master/packages/node_modules/@node-red/nodes/core/function/15-change.html https://github.com/node-red/node-red/blob/master/packages/node_modules/@node-red/nodes/core/function/10-switch.html

Both the built-in switch and change nodes have an entry where you can add multiple properties. I figured this would work nicely for multiple overlays but just couldn't find the time to get it working correctly.