search

peter-murray / node-hue-api

Node.js Library for interacting with the Philips Hue Bridge and Lights
Apache License 2.0
1.19k stars 145 forks source link

Fix for issue #29 #30

Closed knsv closed 9 years ago

knsv commented 9 years ago

Here is the pull request mentioned in issue #29.

I have tried to continue with your patterns. I did add some helper functions for recalling a scene that is not strictly necessary but might result in code that is easier to understand. I did update the docs in this commit also, not sure if you would prefer to have that for the release instead but there it is anyway. As a advocate of automated testing I am sad to say that I have not added any for this functinality. I could not get the suite going fully before my additions so did not want to mess things up.

I hope this addition still adds some value. Use it if you like and I will not mind if you move things around or throws this away.. :)

Cheers!

peter-murray commented 9 years ago

I have pulled in parts of this pull request in the development branch for version 1.0.0, which merges with my refactoring and improvement work.