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

Rewrite of the plugin to solve common issues #4

Merged
merged 10 commits into from
Nov 19, 2016
Merged

Conversation

jochem725
Copy link
Owner

@jochem725 jochem725 commented Oct 13, 2016

Complete rewrite of the plugin to solve some common issues.

  • More robust control flow (resolves connection issues). (Cannot set temperature using Homebridge #2, Losing Toon connection #3)
  • Requesting data now has the same behaviour as the Toon op Afstand app.
  • The plugin now correctly notifies Homebridge of changes, instead of waiting for Homebridge to request an update.
  • The plugin can no longer have a cooling state, this has been replaced by off and auto.

npm install -g jochem725/homebridge-toon#toon-refactor

@jochem725 jochem725 mentioned this pull request Oct 13, 2016
@jochem725 jochem725 merged commit 5c34e66 into master Nov 19, 2016
@jochem725 jochem725 deleted the toon-refactor branch November 19, 2016 16:04
@astrapowerrr
Copy link

Hello Jochem,

Busy getting my first Linux experience to get toon working with Siri.
But now I am having trouble with the setup.
It is giving error on wrong credentials? What am I missing?
Username and password are correct

Can you please help me out?

@jochem725
Copy link
Owner Author

Hey @astrapowerrr,

If the same credentials work at https://toonopafstand.eneco.nl/, there might be an issue with the plugin. I've seen this behaviour before in Issue #8, it may occur if someone moves to a different address. A solution is also proposed there. Please open a separate issue if this is not the problem, so we can try to find a solution.

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.

None yet

2 participants