Skip to content

Upgrade blinkpy to re-enable motion detection (Fixes #20380)#20651

Merged
rohankapoorcom merged 1 commit into
home-assistant:devfrom
fronzbot:blink-fix-motion
Feb 1, 2019
Merged

Upgrade blinkpy to re-enable motion detection (Fixes #20380)#20651
rohankapoorcom merged 1 commit into
home-assistant:devfrom
fronzbot:blink-fix-motion

Conversation

@fronzbot
Copy link
Copy Markdown
Contributor

@fronzbot fronzbot commented Feb 1, 2019

Description:

Motion detection now works again, and the pesky warning in the logs about requiring an app update have been eliminated.

In addition, I fixed the logging problems so now logs can be filtered according to the logger component. Ex: you can add blinkpy.api: error to show only errors from that module in the library, which is how everything should have worked from the beginning.

Related issue (if applicable): fixes #20380

Checklist:

  • The code change is tested and works locally.
  • There is no commented out code in this PR.

If the code communicates with devices, web services, or third-party tools:

  • New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

@dgomes

@ghost ghost added the in progress label Feb 1, 2019
@rohankapoorcom rohankapoorcom added this to the 0.86.5 milestone Feb 1, 2019
Copy link
Copy Markdown
Member

@rohankapoorcom rohankapoorcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@rohankapoorcom rohankapoorcom merged commit 25e1639 into home-assistant:dev Feb 1, 2019
@ghost ghost removed the in progress label Feb 1, 2019
@dgomes dgomes modified the milestones: 0.86.5, 0.87.0 Feb 1, 2019
@fronzbot fronzbot deleted the blink-fix-motion branch February 1, 2019 12:01
@balloob balloob mentioned this pull request Feb 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Post 86.1 upgrade with Blink Could not retrieve videos. Message: An app update is required

5 participants