mattdavis90 / node-red-contrib-tado-client

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

TADO AC / Swing "OFF" gives "not allowed" error #71

Closed RedOctoberSUI closed 2 days ago

RedOctoberSUI commented 2 days ago

Hi

First of all thank you very much for the great integration. Really the only way i can use Tado - as out of the box the AC Controller doesnt really work for us. Thanks again!

Im not sure if this is a bug or i am to daft to set correctly:

When i set to heat i set Swings to "OFF" image

However i then get an error that swings are not allowed. image

I would have expected that Swing "OFF" solves the problem, but it does not :)

Thanks!!

mattdavis90 commented 2 days ago

Hi,

I'm glad the node is helpful for you.

Can you confirm which version of the node you're using? I'm pretty certain this has been raised before and I had hoped it had been fixed.

If you can log in to https://app.tado.com and create an overlay in the web app that does what you want then it should be replicable here. If you can get this working and you're comfortable using Chrome (or Firefox) DevTools then it would be great to see what API calls are being made by the app, particularly what is being send as the payload for the call to /overlay

Thanks

RedOctoberSUI commented 2 days ago

Wow, Matt - thank you for the very fast reply!

image

I am on 0.11.0

Sorry, but im an absolute noob re DevTools. I can open them, but no idea what i am looking for and where :(

image

If you want to point me in the right direction, i can certainly try...

mattdavis90 commented 2 days ago

Oh wow, I've never seen the overlay for an AC before - you get a lot of options.

Thanks for giving it a go. In Devtools if you open the "Network" tab, then set up the overlay you should see something like the following.

Screenshot From 2024-11-26 20-06-51

If you can click on the call to overlay then find the data in the payload tab, that's pretty much what I'm after. That should help work out what the Tado API is expecting. Thanks

RedOctoberSUI commented 2 days ago

Much appreciated directions! image

Seems much aligned with your settings - except for the horizontal swing. seems it really doesnt like that one :)

mattdavis90 commented 2 days ago

Oh, interesting. You don't have a horizontalSwing at all. Ok. I'll see if I can work out some logic to auto remove that field

RedOctoberSUI commented 2 days ago

Oh, interesting. You don't have a horizontalSwing at all. Ok. I'll see if I can work out some logic to auto remove that field

much appreciated!

mattdavis90 commented 2 days ago

I've push v0.11.1 to npm - sometimes it can take NodeRed a little time to pick it up

mattdavis90 commented 2 days ago

The new version calls getZoneCapabilities and checks whether horizontalSwing or verticalSwing is present in the FAN or AUTO mode before setting either property - hopefully that works for you

RedOctoberSUI commented 2 days ago

Works like a charm.. thank you so much! How can i buy you a beer?

mattdavis90 commented 2 days ago

Woop woop 🥳

No problem, glad to help!