Skip to content

docs(specs): narrow the event reminder windows - #208

Merged
themightychris merged 1 commit into
developfrom
docs/reminder-windows
Oct 14, 2026
Merged

themightychris merged 1 commit into
developfrom
docs/reminder-windows

Conversation

@themightychris

Copy link
Copy Markdown
Member

Spec fix before plan events-reminders starts: the reminder due-windows as written contradicted the sentence after them (an event created three days out would have received a "one week" reminder). Each reminder is now due only in a band just after its mark: week = [start−7d, start−6d), day = [start−24h, start−18h).

🤖 Generated with Claude Code

https://claude.ai/code/session_01WVK3JSficcyHViXj36opqV

The earlier rule ("due from 7 days out until 1 day out") would send a
"one week" reminder to an event created three days ahead, which the same
section said must not happen. Each reminder is now due only in the day
(week) or six hours (day) after its mark; missing the band skips it.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WVK3JSficcyHViXj36opqV
@themightychris
themightychris merged commit 635ee6b into develop Oct 14, 2026
1 check passed
@themightychris
themightychris deleted the docs/reminder-windows branch October 14, 2026 03:40
@themightychris themightychris mentioned this pull request Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant