fix(reminders): show the day for reminders on another day (#46) #75

makiolaj · 2026-07-13T15:27:14Z

makiolaj commented

2026-07-13 15:27:14 +00:00

Resolves [#46].

Reminders fired ahead of an event on a different day showed only the time
(e.g. 09:30 – 10:00), making it look like the event was happening today.

reminderTimeText now prefixes timed events with a relative day:

today → time only (unchanged)
tomorrow / yesterday → Tomorrow, … / Yesterday, …
another day this week → localized short weekday (Thu, …)
further out → exact medium date (Jul 19, 2026, …) — a bare weekday would be ambiguous

The this-week boundary honours the user's week starts on setting: the resolved
first day of week is threaded through from ReminderNotifier, so e.g. a Sunday reads
as next week under a Sunday-start locale. All-day events keep their explicit date
(never ambiguous); cross-midnight timed events keep both explicit dates.

Testing

New JVM cases in ReminderTimeTextTest (tomorrow, yesterday, this-week weekday,
next-week date, and a week-start-setting boundary case). ./gradlew test lint green.
Verified on the emulator by triggering real reminders:
- Tomorrow, 5:06 PM – 5:36 PM
- Thu, 5:10 PM – 5:40 PM (this week)
- Jul 19, 2026, 5:10 PM – 5:40 PM (next week, under en-US Sunday-start)

CHANGELOG updated under Unreleased → Fixed.

🤖 Generated with Claude Code

Resolves [#46]. Reminders fired ahead of an event on a different day showed only the time (e.g. `09:30 – 10:00`), making it look like the event was happening today. `reminderTimeText` now prefixes timed events with a relative day: - **today** → time only (unchanged) - **tomorrow / yesterday** → `Tomorrow, …` / `Yesterday, …` - **another day this week** → localized short weekday (`Thu, …`) - **further out** → exact medium date (`Jul 19, 2026, …`) — a bare weekday would be ambiguous The this-week boundary honours the user's **week starts on** setting: the resolved first day of week is threaded through from `ReminderNotifier`, so e.g. a Sunday reads as *next* week under a Sunday-start locale. All-day events keep their explicit date (never ambiguous); cross-midnight timed events keep both explicit dates. ### Testing - New JVM cases in `ReminderTimeTextTest` (tomorrow, yesterday, this-week weekday, next-week date, and a week-start-setting boundary case). `./gradlew test lint` green. - Verified on the emulator by triggering real reminders: - `Tomorrow, 5:06 PM – 5:36 PM` - `Thu, 5:10 PM – 5:40 PM` (this week) - `Jul 19, 2026, 5:10 PM – 5:40 PM` (next week, under en-US Sunday-start) CHANGELOG updated under Unreleased → Fixed. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

makiolaj added 1 commit 2026-07-13 15:27:15 +00:00

fix(reminders): show the day for reminders on another day (#46 )

CI / ci (pull_request) Successful in 5m31s

Details

Translations / check (pull_request) Successful in 5s

Details

38a35be0f0

A reminder fired ahead of an event on a different day showed only the
event's time (e.g. "09:30 – 10:00"), making it look like it was
happening today. reminderTimeText now prefixes timed events with a
relative day: "Tomorrow"/"Yesterday", the short weekday for another day
this week, or the exact date for anything further out.

The this-week boundary honours the user's "week starts on" setting: the
resolved first day of week is threaded through from ReminderNotifier, so
e.g. a Sunday reads as next week under a Sunday-start locale. All-day
events keep their explicit date (never ambiguous), and cross-midnight
timed events keep both explicit dates.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

makiolaj merged commit 23e9a5a35b into release/v2.15.0

2026-07-13 15:35:03 +00:00

makiolaj deleted branch fix/reminder-day-context

2026-07-13 15:35:03 +00:00

makiolaj referenced this issue from a commit

2026-07-13 15:35:04 +00:00

Merge pull request 'fix(reminders): show the day for reminders on another day (#46)' (!75) from fix/reminder-day-context into release/v2.15.0

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: makiolaj/calendula#75