Turn on/off "all" the lights

[ghost]

Siri recognizes that i am asking my my hue lights, but doesn't recognize that command by default. In return I get " Sorry, I could not find a light by that name". That command comes as a default command which i cannot seem to get to work. Any fix?

[chr1s1]

Hi!

What is the name of the light?
Have you tried turning on/off all lights? Does it work?

[ghost]

It turns off individual lights when asked. For example "turn off the hallway light" and it does so. It's when I ask "turn off all the lights" it tells me "I could not find a light by that name"

Any suggestions?

Regards,
Jason Bassett
Founder and President

On May 30, 2013, at 11:24 PM, Christoph Weil notifications@github.com wrote:

Hi!

What is the name of the light?
Have you tried turning on/off all lights? Does it work?

—
Reply to this email directly or view it on GitHub.

[chr1s1]

Have you tried "turn of all lights"? It may is a wrong in the readme file.

[ghost]

Hey:) that worked !

Thanks bud. Might want to update the read me file so others aren't confused.

I also have another quick question if you don't mind..

Regards,
Jason Bassett
Founder and President

On May 31, 2013, at 1:00 AM, Christoph Weil notifications@github.com wrote:

Have you tried "turn of all lights"? It may is a wrong in the readme file.

—
Reply to this email directly or view it on GitHub.

[chr1s1]

Sure, just ask ;)

[ghost]

How can I group the lights by the room and have Siri turn off a group of lights? For example, "Siri, turn off my bedroom lights" and she turns off only the lights in that said room.

Seems tricky :(

Regards,
Jason Bassett
Founder and President

On May 31, 2013, at 8:07 AM, Christoph Weil notifications@github.com wrote:

Sure, just ask ;)

—
Reply to this email directly or view it on GitHub.

[chr1s1]

You just need to set the state of a group you defined in the app or elsewhere: http://developers.meethue.com/2_groupsapi.html#25_set_group_state

I had no time to try it, but it should be easy to implement.

I hope I answered your questions ;-)

[ghost]

Hmm seems simple, but.. where do I place the API requests?

On Fri, May 31, 2013 at 10:42 AM, Christoph Weil
notifications@github.comwrote:

You just need to set the state of a group you defined in the app or
elsewhere:
http://developers.meethue.com/2_groupsapi.html#25_set_group_state

I had no time to try it, but it should be easy to implement.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/8#issuecomment-18760103
.

[interstateone]

Creating groups is not a part of the current Hue API, but is slated to be added in the future: http://developers.meethue.com/2_groupsapi.html#22_create_group

I'd like to keep the plugin inline with what the API offers instead of adding client-side implementations, so for now what you're trying to do isn't possible. In the future when the API is updated I would like to have this added to match, as it would definitely be useful.

On 2013-05-31, at 2:13 PM, jasonsykoo notifications@github.com wrote:

Hmm seems simple, but.. where do I place the API requests?

On Fri, May 31, 2013 at 10:42 AM, Christoph Weil
notifications@github.comwrote:

You just need to set the state of a group you defined in the app or
elsewhere:
http://developers.meethue.com/2_groupsapi.html#25_set_group_state

I had no time to try it, but it should be easy to implement.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/8#issuecomment-18760103
.

—
Reply to this email directly or view it on GitHub.

[ghost]

Ah so that being said I would not be able to have siri turn off a number of
lights in a given room.. blehhh

On Fri, May 31, 2013 at 1:20 PM, Brandon Evans notifications@github.comwrote:

Creating groups is not a part of the current Hue API, but is slated to be
added in the future:
http://developers.meethue.com/2_groupsapi.html#22_create_group

I'd like to keep the plugin inline with what the API offers instead of
adding client-side implementations, so for now what you're trying to do
isn't possible. In the future when the API is updated I would like to have
this added to match, as it would definitely be useful.

On 2013-05-31, at 2:13 PM, jasonsykoo notifications@github.com wrote:

Hmm seems simple, but.. where do I place the API requests?

On Fri, May 31, 2013 at 10:42 AM, Christoph Weil
notifications@github.comwrote:

You just need to set the state of a group you defined in the app or
elsewhere:
http://developers.meethue.com/2_groupsapi.html#25_set_group_state

I had no time to try it, but it should be easy to implement.

—
Reply to this email directly or view it on GitHub<
https://github.com/interstateone/siriproxy-hue/issues/8#issuecomment-18760103>

.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/8#issuecomment-18769307
.

[chr1s1]

Naww.. Sorry I missed that first lines..

[interstateone]

I've changed the matching patterns in the latest commit so that both "Turn on/off all lights" and "Turn on/off all the lights" will work, and doesn't require #9. Thanks @chr1s1 for changing the readme, but I thought that making the optional "the" work would be more ideal.

As a note, Rubular was helpful in determining the correct pattern as it shows the captured matches.