Notification system #3216
Notification system #3216
Conversation
|
I have added the command to the |
|
Yes notifications for all of those will be good. Thanks! |
Head branch was pushed to by a user without write access
|
Fixed all the issues, please verify! |
|
Also added the checkin_reminder_notification command in the run_daily schedule |
|
@DonnieBLT Can this please be merged since it frequently gets merge conflicts due to the model addition |
| def handle(self, *args, **options): | ||
| cutoff_date = now().date() - timedelta(days=1) | ||
|
|
||
| users_without_checkin = User.objects.exclude(dailystatusreport__date__gte=cutoff_date) |
There was a problem hiding this comment.
we would only want to show checkin reminders to users who belong to an organization that has check ins turned on.
There was a problem hiding this comment.
The organization model does not appear to have any attribute that indicates whether it has enabled check-ins. Could you please guide me on how to identify such organizations?
There was a problem hiding this comment.
it would be to join the DailyStatusReport to the organization so that daily status reports would only be sent to organizations that had reminders enabled. This would involve a migration.
|
I merged the conflict, hopefully that was right. |
|
Thanks a lot for that! I'll make sure to finish this up as soon as possible |
fixes #2560
A notification system is created for the users that looks like this:
screen-capture (1).webm
However, in order to fully close this issue we need to create a cron job via some scheduler to send these reminders to users.
If you can also review #3209 then it can be easily incorporated