Skip to content

Comments

add 30 seconds to end_time of CameraEvent. Fixes #92 and #154 ... sorta#166

Merged
jkoelker merged 2 commits intojkoelker:masterfrom
snicker:fix_event_detection_for_nest_cams
Jan 18, 2019
Merged

add 30 seconds to end_time of CameraEvent. Fixes #92 and #154 ... sorta#166
jkoelker merged 2 commits intojkoelker:masterfrom
snicker:fix_event_detection_for_nest_cams

Conversation

@snicker
Copy link
Collaborator

@snicker snicker commented Jan 16, 2019

No description provided.

@snicker
Copy link
Collaborator Author

snicker commented Jan 16, 2019

Per some of the feedback in #92, the only way I can think to improve this is to only add the 30 seconds to the end date based on the camera model (Nest Hello, Outdoor IQ) but I only have the Hello so I can't nail down the model for all affected cams. Merge at will!

This will need to be added to a version posted to PyPi in order to get pulled into HomeAssistant to fix issues similar to
home-assistant/core#20133

@jkoelker
Copy link
Owner

\o/ I'll upload a release here shortly, I'm going to send you a request to be a collaborator on the repo as well.

@jkoelker jkoelker merged commit 446074a into jkoelker:master Jan 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants