search

nonsleepr / ha-eufy-security

Custom integration of Home Assistant with EufySecurity
169 stars 44 forks source link

Changes/Improvements #3

Open FuzzyMistborn opened 4 years ago

FuzzyMistborn commented 4 years ago

1) Instead of a service to enable/disable the motion sensor, I'd suggest a switch that can easily be toggled from the UI 2) Other things just looking through the options we should track down/support: chime state (guessing this toggles the doorbell chime, ir mode, doorbell_led_night_mode. Not sure there's anything else. 3) With Skybell there was a motion_triggered sensor that would ping when the doorbell detected activity. Not sure that's possible looking at the settings/options but would be nice to have. 4) A few things maybe could be sensors in the UI. Like SD card space used or whether there's an update available.

FuzzyMistborn commented 4 years ago

Now that I think more about 1 I realize that's not scaleable as with multiple cameras there'd be a ton of switches. I keep thinking of this based on me only having the doorbell, not the other cameras. I do think there's a general preference by HASS devs to put specific services in it's own domain. IE instead of using the camera domain for camera.enable_motion_detection we should do eufy.camera_enable_motion.