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

[enhancement]: Is it possible to expose the Homebase alarm in HomeKit? #26

Closed
billbazza opened this issue May 13, 2022 · 23 comments
Closed
Assignees
Labels
enhancement New feature or request implemented in beta / next release this is implemented in current beta and will be introduced in next release

Comments

@billbazza
Copy link

Describe the solution you'd like

Would this be possible as it would be useful with the home and away settings and the camera motion sensors combined with other HomeKit devices such as window and door sensors.

@billbazza billbazza added the enhancement New feature or request label May 13, 2022
@github-actions
Copy link

Did you check this Road Map pages ?

@lenoxys
Copy link
Contributor

lenoxys commented May 15, 2022

Hello this is already the case station is exposed. I've one in my setup.

lenoxys added a commit that referenced this issue May 15, 2022
@billbazza
Copy link
Author

The hub is exposed with the home and away switches but I can see no alarm
2E3A8BEB-9D0B-4DF8-A367-4C2601C4BD52

@lenoxys
Copy link
Contributor

lenoxys commented May 16, 2022

When enabling debug mode do you see some issue appear?

@lenoxys lenoxys closed this as completed May 16, 2022
@lenoxys lenoxys reopened this May 16, 2022
@lenoxys lenoxys added bug Something isn't working and removed enhancement New feature or request labels May 16, 2022
@lenoxys lenoxys changed the title [enhancement]: Is it possible to expose the Homebase alarm in HomeKit? [bug]: Is it possible to expose the Homebase alarm in HomeKit? May 16, 2022
@github-actions
Copy link

Did you check this Common Issues pages ?

@billbazza
Copy link
Author

When enabling debug mode do you see some issue appear?

No, the homebase appears with the home, away & night setting just fine. What I was asking for it if the inbuilt siren could be exposed as an additional option so that we could use it in HomeKit automations to create a security system.

@lenoxys
Copy link
Contributor

lenoxys commented May 16, 2022

haaa got it now. Dunno if we can trigger that but bear with us I've few time to spend on the plugin as of now.

@lenoxys lenoxys added enhancement New feature or request and removed bug Something isn't working labels May 16, 2022
@github-actions
Copy link

Did you check this Road Map pages ?

@billbazza
Copy link
Author

haaa got it now. Dunno if we can trigger that but bear with us I've few time to spend on the plugin as of now.

Great stuff, this would be amazing.

@thieren
Copy link
Contributor

thieren commented May 25, 2022

Hi @billbazza

Just checked this. Yeah, the eufy-security-client does expose a method to trigger the alarm. But for an implementation some details would have to be clarified:

  • how exactly should the alarm get triggered. The way I see it, there are two options (I prefer option 1):
  1. Integrate via On/Off Switch, so On -> the alarm is triggered: Off -> the alarm is turned off
    Little Drawback: Users could hit the button by accident and would be somewhat confused by the result
  2. Add a configuration option, that the alarm is automatically triggered when specific events are triggered (e.g. motion is detected, so alarm is triggered)
    Drawbacks: Much less control over the automation
  • the plugin has to know for how long the alarm should be triggered. Seems to me, that the alarm cannot run indefinitely. Unfortunately Homekit doesn't have a Characteristic with which you could set a time duration (at least to my knowledge). I think this would be best achieved by a plugin setting in the configuration. Other ideas?

@lenoxys I believe I would have some time in the coming weeks to get this feature running if you don't mind.

edit: Just to clarify: I could trigger the alarm (sound and so on) but didn't get a notification on my phone from the eufy App. I'm not sure if this would be the normal behavior since I don't use the alarm in general. But I think this is good to know.
But as far as I know you should be able to let HomeKit notify you.

@billbazza
Copy link
Author

In the eufy app, you can trigger the alarm in an automation that you can configure from homebase i.e. motion detected while away, sound alert and notify. In the home app, you would need an alarm off switch off another way to deactivate the alarm i.e. a Hidden button press or along down press on a hue switch et cetera. You could even use one of those automations to say if door is unlocked (smart door lock) the alarm will turn off. It would need some careful consideration though.

@thieren
Copy link
Contributor

thieren commented May 29, 2022

Hey @billbazza

Is it possible that you write me up on discord?

It should be no problem to include a button to turn the alarm off.
Theoretically also turning on will be no problem. But I lack some information as to how this could be useful (because I see no way to integrate this into any automations).

@thieren thieren changed the title [bug]: Is it possible to expose the Homebase alarm in HomeKit? [enhancement]: Is it possible to expose the Homebase alarm in HomeKit? May 29, 2022
@thieren thieren self-assigned this May 29, 2022
@sroettgermann
Copy link

I'm interested in this as well. Would be great to have an option to trigger the alarm siren of the homebase via an automation. E.g. some motion gets detected during "away" mode and this leads to an alarm sound.

@thieren
Copy link
Contributor

thieren commented Jun 22, 2022

I'm interested in this as well. Would be great to have an option to trigger the alarm siren of the homebase via an automation. E.g. some motion gets detected during "away" mode and this leads to an alarm sound.

Still working on this.

during "away" mode

That's exactly the problem. In HomeKit automations you cannot check for different modes. So the plugin will have to handle this via an optional setting I think.
But then you'll have a switch which behaviour is not at all obvious to the user since it sometimes does something, and other times not...
It's a bit tricky to implement this with ideal UX.

@sroettgermann
Copy link

Ah okay. I got it. Thanks anyway for your quick answer. Looking forward to the results.

@thieren
Copy link
Contributor

thieren commented Jul 11, 2022

This is implemented in the latest beta. Please give some feedback if you test this. Thx!

@thieren thieren added the implemented in beta / next release this is implemented in current beta and will be introduced in next release label Jul 11, 2022
@sroettgermann
Copy link

The beta is not working with my HB setup. :(

[12/07/2022, 09:42:46] TypeError: device.getStationSerial is not a function at /usr/local/lib/node_modules/homebridge-eufy-security/node_modules/eufy-security-client/src/eufysecurity.ts:517:56 at Array.forEach (<anonymous>) at /usr/local/lib/node_modules/homebridge-eufy-security/node_modules/eufy-security-client/src/eufysecurity.ts:516:21 at processTicksAndRejections (node:internal/process/task_queues:96:5) [12/07/2022, 09:42:46] Got SIGTERM, shutting down Homebridge... [7/12/2022, 9:42:46 AM] [EufySecurity-2.1.0-beta.7] INFO: Finished shutdown! (node:31067) PromiseRejectionHandledWarning: Promise rejection was handled asynchronously (rejection id: 2) (Use node --trace-warnings ... to show where the warning was created)

@thieren
Copy link
Contributor

thieren commented Jul 12, 2022

Hi @sroettgermann

sorry to hear that. bropat changed a few things on the way devices are added and maybe this is related to an issue that will be resolved in his 2.1.1, but to know for sure I'd like to have a look at your logs, if that's possible.

Please enable debug messages and send me the log at /var/local/lib/homebridge/eufysecurity/log-lib.log

@sroettgermann
Copy link

@thieren thank you. I'll have a look at it.

It seems an additional IndoorCam is causing this issue. :-(
I'll send you the log asap.

@sroettgermann
Copy link

@thieren, is it possible to send it to you in a DM somehow?

@thieren
Copy link
Contributor

thieren commented Jul 13, 2022

@thieren, is it possible to send it to you in a DM somehow?

You could send it to me via discord. You'll find the discord link on the main page of this repo.

@thieren
Copy link
Contributor

thieren commented Jul 13, 2022

Alternatively I'll send you a mail.

@thieren thieren mentioned this issue Sep 3, 2022
@thieren
Copy link
Contributor

thieren commented Sep 6, 2022

Is implemented with release of 2.1.0

@thieren thieren closed this as completed Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request implemented in beta / next release this is implemented in current beta and will be introduced in next release
Development

No branches or pull requests

4 participants