Good catch.  This shows a flaw in my work flow.  I omitted a change that I consider part of i2CS dev where I added a new command.  I didn’t realize that I was omitting the renaming of that command as well.

 

From: krkeegan [mailto:notifications@github.com] 
Sent: Monday, February 11, 2013 12:01 AM
To: hollie/misterhouse
Cc: Michael Stovenour
Subject: Re: [misterhouse] Cleaned up some logging and renamed PLM "log links" for consistency (#71)

 

In lib/Insteon.pm:

> @@ -301,7 +301,7 @@ sub generate_voice_commands
>             $object_string .= "$object_name_v -> tie_event('$object_name->complete_linking_as_responder','complete linking as responder');\n\n";
>             $object_string .= "$object_name_v -> tie_event('$object_name->initiate_unlinking_as_controller','initiate unlinking');\n\n";
>             $object_string .= "$object_name_v -> tie_event('$object_name->cancel_linking','cancel linking');\n\n";
> -           $object_string .= "$object_name_v -> tie_event('$object_name->log_alllink_table','show link table to log');\n\n";
> +           $object_string .= "$object_name_v -> tie_event('$object_name->log_alllink_table','log links');\n\n";

I think you need to change the cmd_state on line 299 as well in order for this to work.

—
Reply to this email directly or view it on GitHub <https://github.com/hollie/misterhouse/pull/71/files#r2957530> .. 

  <https://github.com/notifications/beacon/Nvoh7bM6g31WX2NN_lw41aw6CQu8FlPOA0Yz7ZizF67PUMyIBvwKheDfowXkGvVK.gif>