-
Notifications
You must be signed in to change notification settings - Fork 107
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
Add support for motion detection switch #241
Conversation
Codecov Report
@@ Coverage Diff @@
## master #241 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 11
Lines 1373 1373
=========================================
Hits 1373 1373
Continue to review full report at Codecov.
|
As long as we are handling the state topic correctly in frigate, I think it will just toggle back to on by itself. |
…gration into motion-switch
This looks good to me (awaiting parent PR to be merged). |
…gration into motion-switch
@dermotduffy parent PR has been merged 👍 |
…gration into motion-switch
Thanks @NickM-27 ! |
This adds support for blakeblackshear/frigate#3117
Concerns
Only concern is the req
motion detection toggle to off will fail if detect is enabled
. Assuming this req is what is merged, if in HA we call publish motion OFF and it fails on frigate side, will the switch here get set back to the correct state or will it show OFF incorrectly?Basically: do we need to implement this restriction here as well?