diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-03-23 00:42:44 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-03-23 00:42:44 +0000 |
| commit | 6c767194d9470b368f8d337e0719795f235f683c (patch) | |
| tree | 9ffbef1dfa45add88a821877a2e6c8ceb94f52f8 /internal/models | |
| parent | 8abc63efdbc0bb96cd6c9aa99d6e9166e0bcabae (diff) | |
fix: parse Todoist local datetimes, show near-future tasks, add undated tasks to timeline
- parseDueDate: handle date field containing "YYYY-MM-DDTHH:MM:SS" (local time,
no tz offset) — Todoist REST API v1 uses this format for recurring tasks with
a set time, causing due dates to silently parse as nil
- IsFuture threshold: widen from tomorrow to 7 days out so tasks due this week
show in the main tasks section with dates visible (not collapsed)
- BuildTimeline: include undated Todoist tasks in the Today section (mirrors
existing Google Tasks behavior)
- GetUndatedTasks: new store method for tasks with due_date IS NULL
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/models')
| -rw-r--r-- | internal/models/atom.go | 6 | ||||
| -rw-r--r-- | internal/models/atom_test.go | 34 |
2 files changed, 37 insertions, 3 deletions
diff --git a/internal/models/atom.go b/internal/models/atom.go index 9c519ba..3745917 100644 --- a/internal/models/atom.go +++ b/internal/models/atom.go @@ -57,15 +57,15 @@ func (a *Atom) ComputeUIFields() { tz := config.GetDisplayTimezone() now := config.Now() today := time.Date(now.Year(), now.Month(), now.Day(), 0, 0, 0, 0, tz) - tomorrow := today.AddDate(0, 0, 1) // Check if overdue (due date is before today) dueInTZ := a.DueDate.In(tz) dueDay := time.Date(dueInTZ.Year(), dueInTZ.Month(), dueInTZ.Day(), 0, 0, 0, 0, tz) a.IsOverdue = dueDay.Before(today) - // Check if future (due date is after today) - a.IsFuture = !dueDay.Before(tomorrow) + // Check if future (due date is 7+ days out — collapse in tasks tab) + sevenDaysOut := today.AddDate(0, 0, 7) + a.IsFuture = !dueDay.Before(sevenDaysOut) // Check if has set time (not midnight) a.HasSetTime = dueInTZ.Hour() != 0 || dueInTZ.Minute() != 0 diff --git a/internal/models/atom_test.go b/internal/models/atom_test.go index 3ed4774..70bc14b 100644 --- a/internal/models/atom_test.go +++ b/internal/models/atom_test.go @@ -134,6 +134,40 @@ func TestAtom_ComputeUIFields(t *testing.T) { } }) + // Tasks due within 7 days should NOT be IsFuture (shown in main section with dates) + t.Run("tomorrow is not future", func(t *testing.T) { + tz := time.UTC + now := time.Now().In(tz) + tomorrow := time.Date(now.Year(), now.Month(), now.Day()+1, 0, 0, 0, 0, tz) + atom := Atom{DueDate: &tomorrow} + atom.ComputeUIFields() + if atom.IsFuture { + t.Error("Task due tomorrow should not be IsFuture — it should appear in main tasks section with its date visible") + } + }) + + t.Run("6 days out is not future", func(t *testing.T) { + tz := time.UTC + now := time.Now().In(tz) + sixDays := time.Date(now.Year(), now.Month(), now.Day()+6, 0, 0, 0, 0, tz) + atom := Atom{DueDate: &sixDays} + atom.ComputeUIFields() + if atom.IsFuture { + t.Error("Task due in 6 days should not be IsFuture") + } + }) + + t.Run("8 days out is future", func(t *testing.T) { + tz := time.UTC + now := time.Now().In(tz) + eightDays := time.Date(now.Year(), now.Month(), now.Day()+8, 0, 0, 0, 0, tz) + atom := Atom{DueDate: &eightDays} + atom.ComputeUIFields() + if !atom.IsFuture { + t.Error("Task due in 8 days should be IsFuture") + } + }) + // Test with due date at midnight (no specific time) t.Run("midnight due date", func(t *testing.T) { now := time.Now() |
