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

camera.enable_motion_detection and camera.disable_motion_detection not working for all blink camera #52351

Closed
joe8mofo opened this issue Jun 30, 2021 · 3 comments · Fixed by #56474

Comments

@joe8mofo
Copy link

The problem

Calling camera.enable_motion_detection or camera.disable_motion_detection results in no state change.

What is version of Home Assistant Core has the issue?

core-2021.6.6

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

blink

Link to integration documentation on our website

No response

Example YAML snippet

service: camera.enable_motion_detection
target:
  entity_id:
    - camera.blink_kitchen
    - camera.blink_playroom
    - camera.blink_garage
    - camera.blink_front

Anything in the logs that might be useful for us?

2021-06-30 12:28:43 WARNING (SyncWorker_22) [blinkpy.camera] Method is deprecated as of v0.16.0 and will be removed in a future version. Please use the BlinkCamera.arm property instead.
2021-06-30 12:28:44 WARNING (SyncWorker_46) [blinkpy.camera] Method is deprecated as of v0.16.0 and will be removed in a future version. Please use the BlinkCamera.arm property instead.
2021-06-30 12:28:44 WARNING (SyncWorker_9) [blinkpy.camera] Method is deprecated as of v0.16.0 and will be removed in a future version. Please use the BlinkCamera.arm property instead.
2021-06-30 12:28:44 WARNING (SyncWorker_36) [blinkpy.camera] Method is deprecated as of v0.16.0 and will be removed in a future version. Please use the BlinkCamera.arm property instead.

Additional information

No response

@probot-home-assistant
Copy link

blink documentation
blink source
(message by IssueLinks)

@probot-home-assistant
Copy link

Hey there @fronzbot, mind taking a look at this issue as its been labeled with an integration (blink) you are listed as a codeowner for? Thanks!
(message by CodeOwnersMention)

@DavidRayner
Copy link

DavidRayner commented Aug 17, 2021

I'm seeing exactly the same issue.

@fronzbot

Hopefully this will be fixed by using the new arm property in the blinkpy package: fronzbot/blinkpy#273

i.e.

self._camera.arm = True on L47 here
self._camera.arm = False on L51 here

@github-actions github-actions bot locked and limited conversation to collaborators Sep 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants