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

Multiple Garage Doors #7

Closed
sonnie001 opened this issue Dec 31, 2015 · 18 comments
Closed

Multiple Garage Doors #7

sonnie001 opened this issue Dec 31, 2015 · 18 comments

Comments

@sonnie001
Copy link

I am attempting to get 2 garage doors configured and am hitting a wall. The error message received is:

[Single Garage Door] Logged in with user ID 0
[Single Garage Door] WARNING: You have multiple doors on your MyQ account.
[Single Garage Door] WARNING: Specify the ID of the door you want to control using the 'requiredDeviceId' property in your config.json file.
[Single Garage Door] WARNING: You can have multiple liftmaster accessories to cover your multiple doors
[Single Garage Door] Found Door: 1572446 - Double Door
[Single Garage Door] Found Door: 1572447 - Single Door

The Section for myq in my config.json is:

{
"accessory": "LiftMaster",
"name": "Double Garage Door",
"username": "xxxx@xxxx.com",
"password": "xxxxxx",
"deviceID": "1572446-Double Door"
}, {
"accessory": "LiftMaster",
"name": "Single Garage Door",
"username": "xxxxx@xxxx.com",
"password": "xxxxxx",
"deviceID": "1572447-Single Door"
}

I am not getting any errors on launching home bridge until I go to access one of the doors.

Any directions to move past this? I suspect I have entered the syntax of the device ID wrong.

Cheers

@AppleTechy
Copy link
Collaborator

The issue is that you need to change "deviceId" to "requireddeviceId". This should fix your problem

On Dec 31, 2015, at 1:13 PM, sonnie001 notifications@github.com wrote:

I am attempting to get 2 garage doors configured and am hitting a wall The error message received is:

[Single Garage Door] Logged in with user ID 0
[Single Garage Door] WARNING: You have multiple doors on your MyQ account
[Single Garage Door] WARNING: Specify the ID of the door you want to control using the 'requiredDeviceId' property in your configjson file
[Single Garage Door] WARNING: You can have multiple liftmaster accessories to cover your multiple doors
[Single Garage Door] Found Door: 1572446 - Double Door
[Single Garage Door] Found Door: 1572447 - Single Door

The Section for myq in my configjson is:

{
"accessory": "LiftMaster",
"name": "Double Garage Door",
"username": "xxxx@xxxxcom",
"password": "xxxxxx",
"deviceID": "1572446-Double Door"
}, {
"accessory": "LiftMaster",
"name": "Single Garage Door",
"username": "xxxxx@xxxxcom",
"password": "xxxxxx",
"deviceID": "1572447-Single Door"
}

I am not getting any errors on launching home bridge until I go to access one of the doors

Any directions to move past this? I suspect I have entered the syntax of the device ID wrong

Cheers


Reply to this email directly or view it on GitHub.

@danoday
Copy link

danoday commented Dec 31, 2015

Not sure if it is case sensitive, but use "requiredDeviceID" instead of (or in addition to, if you like) "deviceID".

Also, just use the numeric portion of the deviceID. Remove the "-Single Door" and "-Double Door" parts. I don't think those are part of the deviceID itself, only the appended description.

@sonnie001
Copy link
Author

Thanks for the quick responses dandy and AppleTechy... I have modified as suggested, so now my config.json section reads as:

{
"accessory": "LiftMaster",
"name": "Double Garage Door",
"username": "xxx@xxxx",
"password": "xxxx",
"requiredDeviceID": "1572446"
}, {
"accessory": "LiftMaster",
"name": "Single Garage Door",
"username": "xxxx",
"password": "xxxxx",
"requiredDeviceID": "1572447"
}

I am getting the same error in home bridge when accessing the accessories from Insteon+. Now my openers are Chamberlain , not lift master, but I can access the MyQ account through Liftmaster so didn't think that would matter.

@danoday
Copy link

danoday commented Dec 31, 2015

Hmm... have you stopped and restarted Homebridge from the command line. I don't think it recognizes config.json changes until reload.

@sonnie001
Copy link
Author

danoday... the error that is generated from accessing the door accessory crashes Homebridge with:

/usr/local/lib/node_modules/homebridge-liftmaster/index.js:147
throw "FATAL: Please specify which specific door this Liftmaster accessory should control - you have multiples on your account";
^
FATAL: Please specify which specific door this Liftmaster accessory should control - you have multiples on your account

so I have restarted Homebridge with the new config.json but with the same result

@AppleTechy
Copy link
Collaborator

When did you install the plugin? Me and Nfarina updated the plugin about 2 days ago....

Sent from my iPhone

On Dec 31, 2015, at 1:31 PM, sonnie001 notifications@github.com wrote:

danoday... the error that is generated from accessing the door accessory crashes Homebridge with:

/usr/local/lib/node_modules/homebridge-liftmaster/index.js:147
throw "FATAL: Please specify which specific door this Liftmaster accessory should control - you have multiples on your account";
^
FATAL: Please specify which specific door this Liftmaster accessory should control - you have multiples on your account

so I have restarted Homebridge with the new config.json but with the same result


Reply to this email directly or view it on GitHub.

@sonnie001
Copy link
Author

I installed the plugin today, about 2 hours ago.

@danoday
Copy link

danoday commented Dec 31, 2015

Line 147 is a manual abend if it finds multiple doors without a device ID being specified. It shouldn't even be hitting that code in your situation.

I might have had the variable wrong - try "requiredDeviceId" with a lower-case "D" at the end, not "requiredDeviceID" with an upper-case "D". Restart and see if that helps.

@AppleTechy
Copy link
Collaborator

The variable should read "requiredDeviceId" the should solve your problems. Save that, quit terminal and then try reloading homebridge. I believe that should be the correct variable. Or it might be DeviceId but the capitalization does matter! Wish I could be more help on what I have the variable set but I am not in front of my computer right now. Let me know if that works.

On Dec 31, 2015, at 1:42 PM, danoday notifications@github.com wrote:

Line 147 is a manual abend if it finds multiple doors without a device ID being specified. It shouldn't even be hitting that code in your situation.

I might have had the variable wrong - try "requiredDeviceId" with a lower-case "D" at the end, not "requiredDeviceID" with an upper-case "D". Restart and see if that helps.


Reply to this email directly or view it on GitHub.

@sonnie001
Copy link
Author

I have updated the config.json with "requiredDeviceId" and that seams to have fixed the error! Thanks guys!. I shall now play with it and see how it performs before rolling out to my wife.

@AppleTechy
Copy link
Collaborator

The only issue I have is Siri won't allow you to open/close one of the doors. She makes you close/open all of the doors. Let me know if that is the same case for you. You might just end up removing the single garage bay door that I never use. So that way Siri's voice command function with HomeKit is useful with my garage door. Thxs.

On Dec 31, 2015, at 1:50 PM, sonnie001 notifications@github.com wrote:

I have updated the config.json with "requiredDeviceId" and that seams to have fixed the error! Thanks guys!. I shall now play with it and see how it performs before rolling out to my wife.


Reply to this email directly or view it on GitHub.

@danoday
Copy link

danoday commented Dec 31, 2015

@sonnie001 - Great news! You might look at issue #6, and insert the code there to get the door status reporting correctly through Siri.

@AppleTechy - I can help with your Siri problem. Siri defaults each garage door to a Siri name of "Garage Door", instead of "Big Garage Door" and "Small Garage Door"... or whatever you called them. This makes Siri confused about which one you are referring to, and she wants to close/open both. To change this, look for the "Service Name" in an app, which isn't necessarily the "Accessory Name". If you download the Insteon+ app, you'll find this under "Siri Control" when you edit the device.

Let me know if that works.

@sonnie001
Copy link
Author

@danoday silly question (likely!) where is the index.js file located?

@AppleTechy
Copy link
Collaborator

It is located at /usr/local/lib/node_modules/homebridge-LiftMaster
Good luck! @danoday thank you. I had been using the HM catalog app supplied by Apple for when I was setting up homebridge. I recently downloaded Insteon+ and fixed my problem

On Dec 31, 2015, at 2:45 PM, sonnie001 notifications@github.com wrote:

@danoday silly question (likely!) where is the index.js file located?


Reply to this email directly or view it on GitHub.

@AppleTechy
Copy link
Collaborator

close #7

@danoday
Copy link

danoday commented Jan 3, 2016

I don't know that we should close this quite yet. The "requiredDeviceId" thing really needs to be either changed in the code, or altered in the documentation before this issue is truly "closed"... otherwise, people will be opening a new issue for it every time.

Maybe we should create an issue titled "Variable deviceId is incorrect; please use requiredDeviceId instead".

@AppleTechy
Copy link
Collaborator

@danoday i will message @nfarina asking him to change the documentation

@nfarina nfarina closed this as completed in f5a3f2b Jan 6, 2016
@nfarina
Copy link
Owner

nfarina commented Jan 6, 2016

Hey all, sorry about this, this happened because I used a README I created while I was working on a rewrite of this plugin, but then when it was taking too long I pasted in the old code.

I fixed index.js to use the correct parameter - deviceID.

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

No branches or pull requests

4 participants