Skip to content

Add support with ss3 version of SimpliSafe REST APIs - #6

Open
chowielin wants to merge 15 commits into
fholgado:masterfrom
chowielin:master
Open

Add support with ss3 version of SimpliSafe REST APIs#6
chowielin wants to merge 15 commits into
fholgado:masterfrom
chowielin:master

Conversation

@chowielin

@chowielin chowielin commented Apr 8, 2018

Copy link
Copy Markdown

No NPM package published yet. In install.sh, add:
npm install -g git+https://github.com/chowielin/homebridge-simplisafe-security-system.git

@enriquecaballero

Copy link
Copy Markdown

+1

@enriquecaballero

Copy link
Copy Markdown

Just thought I'd throw this out there -- this project hasn't been updated in about 2 years and I have no idea if it's maintained anymore; I wouldn't wait on this getting merged. If no response happens, just release it yourself under another name.

@chowielin

Copy link
Copy Markdown
Author

@enriquecaballero ok yeah I noticed. I will probably go ahead and publish a separate NPM then. I will also look to separate the SS3 client to a separate library so others can use it outside of Homebridge.

@enriquecaballero

Copy link
Copy Markdown

@chowielin Cool. I started looking into maybe getting the separate (motion) sensors to show up in HomeKit as well. Looks like there's an endpoint that returns JSON with all the sensors tied to the system.

@chowielin

Copy link
Copy Markdown
Author

@enriquecaballero cool. I noticed that as well and added that function in the SS3Client called "getSensors". I saw it gives status for doors being opened or not.

To handle case where alarm is triggered and the status was not being reported correctly. ALARM_COUNT doesn't denote whether home mode or away mode was triggered so just defaulting to away mode.
@mbriney

mbriney commented Nov 22, 2018

Copy link
Copy Markdown

It seems the update SimpliSafe did the other day has removed access to the previous APIs. It gets a token but doesn't update status properly. I'd recommend merging in the code from @chowielin as pulling his repository fixes the package.

@ddoria921

Copy link
Copy Markdown

@chowielin are you still thinking about publishing this as a separate npm package?

@chowielin

Copy link
Copy Markdown
Author

@ddoria921 It’s on my radar. I forgot about doing that! Glad people are finding it useful.

@ddoria921

Copy link
Copy Markdown

Ping me when it’s published. I want to contribute and I figured I might as well pitch in on the more recently miantened package. Thanks for keeping it up-to-date!

@fholgado

Copy link
Copy Markdown
Owner

D'oh. I don't know why GitHub isn't sending me emails about this. Sorry y'all, would have totally merged this in sooner :(

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

Successfully merging this pull request may close these issues.

6 participants