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

Fix Nest not responding when it has been set to away mode manually #150

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

philipkocanda
Copy link

@philipkocanda philipkocanda commented Dec 2, 2017

Fixes #75

If I try and change the temperature by the slider, it seems to set it, get no response back, and then seems to set it back to what it was before:

[9/16/2016, 6:09:57 PM] Target temperature for Living Room Thermostat is: 30 C
[9/16/2016, 6:09:57 PM] Target temperature for Living Room Thermostat is: 19 C

This only happened when it was set to away.

The issue here is that the Nest will only allow you to change temperature when it's not in away mode, so we have to make sure it is not.

This only used to be done when it was set to auto-away, but since I (and others) set the away mode through the API or manually set the device to away, this breaks controlling the Nest through homekit.

@philipkocanda
Copy link
Author

After some local testing it looks like the first request still fails, because homebridge-nest doesn't wait for the "set to home" request to complete. Looking into a fix.

@chrisjshull
Copy link

Great news! I have taken up maintenance of the homebridge-nest plugin in npm. The source code now lives at https://github.com/chrisjshull/homebridge-nest/. To update to the latest version of homebridge-nest, simply run npm update -g homebridge-nest.

I believe I've fixed this issue in that repo. If not, please submit a new issue/PR there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants