search

fuatakgun / eufy_security

Home Assistant integration to manage Eufy Security devices as cameras, home base stations, doorbells, motion and contact sensors.
889 stars 73 forks source link

Eufy off/disarmed #890

Closed onelius closed 1 year ago

onelius commented 1 year ago

Describe the bug The alarm shows as off in home assistant instead off disarmed as before.

Expected behavior Off in the eufy app and on eufy keypad should show as disarmed in home assistant.

Disarmed in home assistant should place eufy app and keypad in off mode.

Additional information In version 6.2 it was working as expected, when updated to lates version it changed.

fuatakgun commented 1 year ago

@kvanbiesen , can you chime in so we can all discuss best way forward.

kvanbiesen commented 1 year ago

In my opinion, nothing. Cause off is off and disarmed is disarmed. Since both states exist in the eufy app, imho both states should exist in HA.

It's the opposite of https://github.com/fuatakgun/eufy_security/issues/840 Which is in my option the more valid one.

These user should adapt his automations (if he has any) to take in both both states...

If he wants to see disarmed, he should set his alarm to disarmed. Technically we could say states away, geofence, schedule, custom1 .... As just armed but we don't do that either so yeah off stays.

(Not because I choose, but because it makes more sense, to have the actual state in there)

On Tue, Aug 15, 2023, 23:43 Fuat Akgün @.***> wrote:

@kvanbiesen https://github.com/kvanbiesen , can you chime in so we can all discuss best way forward.

— Reply to this email directly, view it on GitHub https://github.com/fuatakgun/eufy_security/issues/890#issuecomment-1679658204, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZ4APJD5FFQ6VRJF32BURTXVPUQJANCNFSM6AAAAAA3RSKSV4 . You are receiving this because you were mentioned.Message ID: @.***>