Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

[nest] October 2016 API Update #4716

Merged
merged 4 commits into from
Oct 25, 2016
Merged

[nest] October 2016 API Update #4716

merged 4 commits into from
Oct 25, 2016

Conversation

watou
Copy link
Contributor

@watou watou commented Oct 20, 2016

Updates to the Nest binding to match the October 2016 API Update.

Please test this JAR to verify that the new features work. You will have to update your Product at developer.nest.com to use the v6 permissions, and set up a new PIN for your account and update your nest.cfg or openhab.cfg file. Also, the thermostat against which you are testing must be at the 5.6 or later firmware version.

Please report back here or on the forum with your results, and thanks!

@watou watou added this to the 1.9.0 milestone Oct 20, 2016
@watou watou self-assigned this Oct 20, 2016
@mrguessed
Copy link
Contributor

mrguessed commented Oct 23, 2016 •
edited
Loading

Also, the thermostat against which you are testing must be at the 5.6 or later firmware version.

To validate the firmware version of the TStats, you can use the following menu on the device itself:

Settings / Technical Info / Display

You can see the same data from the Nest Application by:

  • Selecting the TStat
  • Selecting the Gear ⚙️ in the upper right hand corner
  • Selecting Technical Info

If your TStats are not (yet) on the 5.6 version, the new binding will crash.

@watou : You may want to add code that validates the software_version field of each TStat, and dump a warning.

Currently you'll bang out with an parsing exception.
The code itself could be temporary, mostly to handle the Nest's cutover transition period - my app updated a few days back, but the TStats still haven't (and I'd assumed they had)

[EDIT] Adding a more specific link to the forum discussion:
https://community.openhab.org/t/nest-binding-updates/15536

@watou
Fixed type of time_to_target
228e3a9 
Signed-off-by: John Cocula <john@cocula.com>
@watou
Copy link
Contributor Author

watou commented Oct 23, 2016

Sorry; here's a new test JAR.

@mrguessed
Copy link
Contributor

Looks better now, thanks for the quick turn-around!

@watou
Copy link
Contributor Author

watou commented Oct 23, 2016

Thanks for testing again, Mark.

@watou
previous_hvac_mode can be an empty string.
4174589 
Signed-off-by: John Cocula <john@cocula.com>
@watou
Copy link
Contributor Author

watou commented Oct 24, 2016

A new test JAR, including new commit that accepts "" for previous_hvac_mode.

@watou
Simplify JSON deserialization for time_to_target_training.
0da29f3 
Signed-off-by: John Cocula <john@cocula.com>
@watou
Copy link
Contributor Author

watou commented Oct 24, 2016

Only based on a guess, but...

511903214 278542 org.openhab.binding.nest-1.9.0-SNAPSHOT.jar

@watou
Copy link
Contributor Author

watou commented Oct 24, 2016

I tested the most recent JAR (linked above) under a recent OH2 nightly and the v6 Nest Product permissions, and my JSON payload is parsing completely. New item bindings from this PR were coming through as expected. The thermostats in the payload are both still at v5.5 firmware.

About testing under OH2: The configuration settings via Paper UI don't seem to have any effect on the running binding. I needed to enter them manually into nest.cfg before the binding would start doing work.

@watou watou merged commit 7918c1a into openhab:master Oct 25, 2016
@watou watou deleted the nest-oct-2016-api-update branch October 25, 2016 17:51
@vzorglub
Copy link
Contributor

vzorglub commented Nov 2, 2016

Hello,
I know you have commited the change of the binding but the version in the Openhab downloads is still 1.8.3. Where can I download the 1.9.0. jar version. Thanks

@watou
Copy link
Contributor Author

watou commented Nov 3, 2016

Where can I download the 1.9.0. jar version.

@vzorglub: For openHAB 1.x users, you can get the new JAR file here. Just replace the JAR in your addons folder.

cdjackson added a commit to cdjackson/openhab1-addons that referenced this pull request Nov 8, 2016
@cdjackson
Merge branch 'master' into dbadia-security-mmerge
b59aaba 
* master: (29 commits)
  Zwave database (openhab#4755)
  Fatek PLC binding (openhab#2784)
  Update PAN08 (openhab#4750)
  Fixed some typos in jdbc log messages. (openhab#4746)
  [Mqttitude] Add the 1.9 binding to the OH2 distro (openhab#4743)
  Change default retention policy name to "autogen" (documentation have to be adapted). Bugfix: honour retention policy in queries. (openhab#4744)
  corrected link to calimero: (openhab#4739)
  TCP/UDP Binding enhancement 'MQTT style' (openhab#4658)
  [nma] Add NotifyMyAndroid actions to OH2 (openhab#4738)
  Add AutoUpdateBindingProvider for OH2 compatibility (openhab#4736)
  Log error returned in request for data model if there was an error. (openhab#4731)
  Bump jdbc postgressql 9.4 version to 1211 and minimal version to 1208. These (openhab#4726)
  [DSMR] Add openHAB 2 feature addon (openhab#4719)
  [nest] October 2016 API Update (openhab#4716)
  Moved binding.xml to correct place per docs. (openhab#4722)
  Fix openhab#366 (openhab#4721)
  Fix MPD handling of invalid volume levels (openhab#4715)
  [mpd] respect playerId in getItemNamesByPlayerAndPlayerCommand (openhab#4713)
  [mios] Add support for UI7 style thermostats. (openhab#4710)
  Database 161016 (openhab#4709)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@watou watou

Labels
enhancement
Projects
None yet
Milestone
1.9.0
Development

Successfully merging this pull request may close these issues.
3 participants
@watou @mrguessed @vzorglub