-
Notifications
You must be signed in to change notification settings - Fork 10
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
#470 Notification Task and Logic #475
Conversation
7092209
to
43fc50e
Compare
logging could be made more extensive. can be done in a futur improvement too.
|
it would be worth modifying the settings_locale template to update the CELERY_BEAT_SCHEDULE including the new task. Also to make sure the recommended schedule executes this task before the email sending task. |
I added some more logging at the Notification object creation time |
Related to #470
Please first apply the migration and then proceed.
You can create the celery task from the admin view and test it with periodic tasks.