fuatakgun / eufy_security

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

Map "Night" to native Home Assistant alarm state instead of Custom 2 #1150

Closed Ged-fi closed 1 month ago

Ged-fi commented 1 month ago

In #145 alarm_night was mapped to Custom 2 in Eufy.

However, Eufy and Home Assistant both "natively" support Night mode so the current mapping is unintuitive and requires creating a Custom alarm mode when Night would work better.

Home Assistant, especially using Mushroom cards will not display "Night" correctly with the current Custom mapping. Neither will Eufy display "Night".

Please reconsider mapping "alarm_night" natively to Night in Eufy.

If the Custom 2 mapping is required for Home Kit for some weird reason, consider adding an option "map night to Custom 2" in the configuration instead.

fuatakgun commented 1 month ago

Thanks for the idea and I agree that it looks unintuitive at first but we ended up in this state after finding out that Night is not a fixed mode across all models. Sharing a picture below from my home base 2.

Please check your home base's night mode if it is deletable, if yes, it is just another custom config so it means not everyone has it on their system Screenshot_2024-05-23-08-02-49-340_com.oceanwing.battery.cam.jpg

Ged-fi commented 1 month ago

Ah, I can confirm that "Night" is actually deletable on my model, so it does qualify as a custom config! As such, I agree with the current slightly un-intuitive implementation.

Sorry for the extra work in looking at this feature request. I suggest it can be closed.