Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tado fallback: false doesn't works! #43118

Closed
energywave opened this issue Nov 11, 2020 · 4 comments · Fixed by #44665
Closed

Tado fallback: false doesn't works! #43118

energywave opened this issue Nov 11, 2020 · 4 comments · Fixed by #44665
Assignees

Comments

@energywave
Copy link

The problem

Tado integration introduced the fallback parameter about one year ago. Doc says the default is true so I set it to false as I wants that manual overrides of zones temperature should be until a different setting is issued and doesn't return to automatic schedule on the next scheduled period.
But it doesn't works, when scheduled time is coming the manual temperature that I've set will be returned to auto and the schedule is returning active.
If I see in the Tado app I can see that all the manual temperatures I set with HA are "until next schedule" (translated from italian...)
Is there a way to let fallback: false to work?

Environment

  • Home Assistant Core release with the issue: 0.117.5
  • Last working Home Assistant Core release (if known):
  • Operating environment (OS/Container/Supervised/Core): HassOS on Raspberry Pi 4 / 4GB
  • Integration causing this issue: Tado
  • Link to integration documentation on our website: https://www.home-assistant.io/integrations/tado/

Problem-relevant configuration.yaml

tado:
  username: !secret tado_username
  password: !secret tado_password
  fallback: false

Traceback/Error logs

Additional information

@probot-home-assistant
Copy link

Hey there @michaelarnauts, @bdraco, mind taking a look at this issue as its been labeled with an integration (tado) you are listed as a codeowner for? Thanks!
(message by CodeOwnersMention)

@Savjee
Copy link
Contributor

Savjee commented Dec 19, 2020

Can confirm. I'm having the same issue. I checked out the Home Assistant component and I believe it's due to a change of Tado's API.

I've documented my findings and opened an issue on the underlying library: wmalgadey/PyTado#41

@energywave
Copy link
Author

Furthermore as I hope that a dev will handle this (let's hope...) it would be very useful to specify fallback with a parameter to the service, not globally in the configuration! Or better, specify a global value like now in configuration but the possibility to specify it even as a parameter in the service. Some automation could so use it, some not, depending on the usage context.

@Savjee
Copy link
Contributor

Savjee commented Dec 22, 2020

Small update on the actual issue. I've published a pull request for the Tado library: wmalgadey/PyTado#44

It's a very simple fix. As soon as this one gets merged in, I can submit a fix for Home Assistant.

@github-actions github-actions bot locked and limited conversation to collaborators Feb 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants