MaddyUnderStars / MaddyUnderStars/strawbean

Repeating reminders first sent in a different tz will have incorrect repeat length

Open
#18 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
TypeScript
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

If a reminder is sent in one tz, and it's notification will be sent in a different tz ( eg daylight savings ), we calculate if this is a into or from DST transition and subtract/add an hour accordingly.

However, for repeating reminders, this code is never run again. And so repeating reminders have their length set an additional hour in either direction to account for that first tz change.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing the reminder scheduling path for repeating reminders and the existing daylight-saving-time adjustment described in the issue. Compare the initial notification calculation with later repeats, then verify that repeats retain the correct duration when the sender and notification time zones cross a DST transition.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.