search

fdegier / homebridge-jablotron-alarm

Homebridge plugin for connecting Jablotron JA-100 alarm systems to Homekit.
The Unlicense
26 stars 6 forks source link

Update installation guide in the README #88

Closed spookercz closed 10 months ago

spookercz commented 1 year ago

In the section of "Identify Jablotron services and devices" you are guiding to run this command:

cd /usr/local/lib/node_modules/homebridge-jablotron node config-helper.js username password

But my homebridge-jablotron is stored elswhere:

Node.js Version: v18.13.0 Node.js Path: /opt/homebridge/bin/node Plugin Path: /var/lib/homebridge/node_modules/homebridge-jablotron

fdegier commented 1 year ago

Thanks for your issue! What do you propose I should do? There are many ways to install Homebridge and plugins, mine is just an example.

spookercz commented 1 year ago

Up to you, I am a newbie to this, so I was looking for straightforward manual. My installation was using the home-bridge raspberry pi default image, so this was the default path. Maybe mention this path as an alternative where to find the right folder with the file. Anyway, thanks for your plugin, really appreciated.

BR Michal

st 18. 1. 2023 v 17:02 odesílatel Fred de Gier @.***> napsal:

Thanks for your issue! What do you propose I should do? There are many ways to install Homebridge and plugins, mine is just an example.

— Reply to this email directly, view it on GitHub https://github.com/fdegier/homebridge-jablotron-alarm/issues/88#issuecomment-1387311410, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5KHTAUJPKQ4AC2L47NAJUTWTAHZ7ANCNFSM6AAAAAAT7FPZVY . You are receiving this because you authored the thread.Message ID: @.***>

misncz commented 1 year ago

@spookercz I have the manual change on my todo list since I was reinstalling using raspbian too. Glad you figured out where the plugin location sits with raspbian!

fdegier commented 1 year ago

Thanks for the feedback both. I actually prefer Docker and have been running HomeBridge in Docker for years now. I should probably make that the default in the README as its 10x simpler than the current implementation.