search

hjdhjd / homebridge-doorbird

Homebridge plugin for Doorbird
Apache License 2.0
54 stars 14 forks source link

minor issue in read.me file #70

Closed axelbrunger closed 4 years ago

axelbrunger commented 4 years ago

Just installed the latest version of the doorbird plugin. It worked great when using the settings in the GUI. However, the read.me file is not complete. The first "name" statement is missing in the read.me file, and homebridge would not start. I then used the GUI to regenerate the statements in the config.json file, and it generates: { "name": "Doorbird", "audio": true, "doorbirds": [ { "name": "DoorBird", "ip": "xxx", "username": "xxx", "password": "xxx" } ], "platform": "Doorbird" }

In other words, there have to be "name" statements in both the "platform" and the "doorbirds" levels.
This configuration works!

hjdhjd commented 4 years ago

Thanks for reporting this, but I can't replicate your error.

You don't need the name field in any portion of the plugin. Perhaps you had a typo in your config.json and that's why it wouldn't start?

axelbrunger commented 4 years ago

Perhaps. Not sure. But the GUI generates both name fields…

On Jun 7, 2020, at 2:37 PM, hjdhjd notifications@github.com<mailto:notifications@github.com> wrote:

Thanks for reporting this, but I can't replicate your error.

You don't need the name field in any portion of the plugin. Perhaps you had a typo in your config.json and that's why it wouldn't start?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/brownad/homebridge-doorbird/issues/70#issuecomment-640283542, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AN3VLYJ52OBKHAUIZMDKGODRVQCANANCNFSM4NXVODHQ.

hjdhjd commented 4 years ago

Thanks. Closing this one out.