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

Alarm motion event database #46

Merged
merged 18 commits into from Aug 9, 2020
Merged

Alarm motion event database #46

merged 18 commits into from Aug 9, 2020

Conversation

mxmaxime
Copy link
Collaborator

@mxmaxime mxmaxime commented Aug 9, 2020

Remaining questions

What to save in database

Please see issue #70

Otherwise you will have the django.core.exceptions.ImproperlyConfigured exception because tasks import models and to do so the app is using INSTALLED_APPS django settings, and thus it's not initialized yet. So we're doing it before imports.
Otherwise the main thread is locked by the DetectMotion thing. So, the mqtt_client doesn't send every x seconds the PINGREQ. Fix issue #45
@mxmaxime mxmaxime added the enhancement Improve internal stuffs label Aug 9, 2020
@mxmaxime mxmaxime added this to the First release milestone Aug 9, 2020
@mxmaxime mxmaxime self-assigned this Aug 9, 2020
@mxmaxime mxmaxime added this to In progress in v0.1.0 via automation Aug 9, 2020
Copy link
Collaborator Author

@mxmaxime mxmaxime left a comment

Choose a reason for hiding this comment

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

Good to go.

@mxmaxime mxmaxime merged commit 22ceaad into master Aug 9, 2020
v0.1.0 automation moved this from In progress to Done Aug 9, 2020
@mxmaxime mxmaxime deleted the alarm-motion-event-database branch August 30, 2020 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve internal stuffs
Projects
No open projects
v0.1.0
  
Done
1 participant