feat: add time zone setting for reminders #1092

Merged
konrad merged 7 commits from feature/timezone-reminders into main 2022-01-16 11:05:58 +00:00
Owner

Description

Instead of naeveily checking for all reminders due in the next minute, we now check all reminders in all time zones in the next minutes. This essentially means checking for reminders due in the next 14 or past 12 hours. We then check for each user who would receive a reminder from that result if it is actually due in their time zone.

This should prevent issues where users would get the reminder in the time zone of their server, not in their own.

Checklist

  • I added or improved tests
# Description Instead of naeveily checking for all reminders due in the next minute, we now check all reminders in all time zones in the next minutes. This essentially means checking for reminders due in the next 14 or past 12 hours. We then check for each user who would receive a reminder from that result if it is actually due in their time zone. This should prevent issues where users would get the reminder in the time zone of their server, not in their own. # Checklist * [X] I added or improved tests
konrad added 1 commit 2022-01-12 21:20:08 +00:00
continuous-integration/drone/pr Build encountered an error Details
da25b40c65
feat: add time zone setting for reminders
Instead of naeveily checking for all reminders due in the next minute,
we now check all reminders in all time zones in the next minutes.
This essentially means checking for reminders due in the next 14 or past
12 hours. We then check for each user who would receive a reminder from
that result if it is actually due in their time zone.

This should prevent issues where users would get the reminder in the
time zone of their server, not in their own.
konrad added 3 commits 2022-01-12 21:46:21 +00:00
konrad added 1 commit 2022-01-14 17:24:48 +00:00
continuous-integration/drone/pr Build is passing Details
812df8401b
fix: lint
konrad added 1 commit 2022-01-14 21:06:04 +00:00
konrad added 1 commit 2022-01-16 10:50:53 +00:00
continuous-integration/drone/pr Build is passing Details
c96729c458
Merge branch 'main' into feature/timezone-reminders
konrad merged commit 61d49c3a56 into main 2022-01-16 11:05:58 +00:00
konrad deleted branch feature/timezone-reminders 2022-01-16 11:05:58 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#1092
No description provided.