benknight / hue-alfred-workflow

An Alfred workflow for controlling Philips Hue lights.
367 stars 34 forks source link

Error During Installation #58

Closed mcgaritydotme closed 3 years ago

mcgaritydotme commented 3 years ago

Using Alfred 4.5, I receive the following error when attempting to connect to my bridge (either using the manual button or explicitly entering the bridge's IP address):

{ "items": [ { "valid": false, "subtitle": "u'username'", "icon": { "path": "/System/Library/CoreServices/CoreTypes.bundle/Contents/Resources/AlertStopIcon.icns" }, "title": "Error in workflow 'Philips Hue Controller'" } ] }

mjl commented 3 years ago

I just had the same experience and tried debugging.

It seems action.py:209 should read

if this_query.startswith('set_bridge:'):

or perhaps

if query[0] == 'set_bridge':

otherwise it will never call the setup function.

benknight commented 3 years ago

Can you guys try downloading the version from the releases page while I debug this? sorry about that.

On Mon, Aug 16, 2021, 16:05 Martin J. Laubach @.***> wrote:

I just had the same experience and tried debugging.

It seems action.py:209 should read

if this_query.startswith('set_bridge:'):

or perhaps

if query[0] == 'set_bridge':

otherwise it will never call the setup function.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/benknight/hue-alfred-workflow/issues/58#issuecomment-899784036, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGACA4CBVQBXDJ5ZZQET2LT5FVRBANCNFSM5BXNQU3Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

benknight commented 3 years ago

The last version*

On Mon, Aug 16, 2021, 20:35 Benjamin Knight @.***> wrote:

Can you guys try downloading the version from the releases page while I debug this? sorry about that.

On Mon, Aug 16, 2021, 16:05 Martin J. Laubach @.***> wrote:

I just had the same experience and tried debugging.

It seems action.py:209 should read

if this_query.startswith('set_bridge:'):

or perhaps

if query[0] == 'set_bridge':

otherwise it will never call the setup function.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/benknight/hue-alfred-workflow/issues/58#issuecomment-899784036, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGACA4CBVQBXDJ5ZZQET2LT5FVRBANCNFSM5BXNQU3Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

mcgaritydotme commented 3 years ago

Thanks for the responses. I got this error using the latest version from the downloads page (3.0.5, from July)

benknight commented 3 years ago

There was a logic bug introduced in #57. I've fixed and published as v3.0.6

mcgaritydotme commented 3 years ago

Tested successfully. Thank you for the turnaround, and great work on this overall!