diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-01-22 10:09:07 -1000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-01-22 10:09:07 -1000 |
| commit | 7fd381a242f68b7c6f10db4e3ae0bb3d06e36a16 (patch) | |
| tree | abff0af0a0f3b057d7b1ad6d95dbefdf30c553c3 /issues/006-reorder-tasks-by-urgency.md | |
| parent | 583f90c5dedf0235fa45557359b0e6e7dd62b0f0 (diff) | |
Fix background image CORS issue
Switch from Unsplash Source API to Lorem Picsum which has proper
CORS headers for cross-origin image loading.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Diffstat (limited to 'issues/006-reorder-tasks-by-urgency.md')
| -rw-r--r-- | issues/006-reorder-tasks-by-urgency.md | 63 |
1 files changed, 63 insertions, 0 deletions
diff --git a/issues/006-reorder-tasks-by-urgency.md b/issues/006-reorder-tasks-by-urgency.md new file mode 100644 index 0000000..8a5f968 --- /dev/null +++ b/issues/006-reorder-tasks-by-urgency.md @@ -0,0 +1,63 @@ +# [FEATURE] Reorder task list by urgency + +## Description +List past due tasks, then tasks with a specific time today, then 12am (all-day) tasks today, then future tasks. + +## User Story +As a user, I want tasks sorted by urgency tiers so I see what needs attention first. + +## Technical Context +- Affects: `internal/handlers/handlers.go` — sorting logic before template render +- Requires distinguishing "has specific time" vs "all-day" (midnight/00:00) +- Todoist API may return all-day tasks with time set to 00:00:00 + +## Test Strategy + +### Unit Test (Red) +**File:** `internal/handlers/handlers_test.go` + +```go +func TestSortTasksByUrgency(t *testing.T) { + tasks := []Task{ + {Name: "future", DueDate: tomorrow}, + {Name: "overdue", DueDate: yesterday}, + {Name: "today-timed", DueDate: todayAt(14, 30)}, + {Name: "today-allday", DueDate: todayAt(0, 0)}, + } + + sorted := SortTasksByUrgency(tasks) + + // Expected order: overdue, today-timed, today-allday, future + assert.Equal(t, "overdue", sorted[0].Name) + assert.Equal(t, "today-timed", sorted[1].Name) + assert.Equal(t, "today-allday", sorted[2].Name) + assert.Equal(t, "future", sorted[3].Name) +} +``` + +## Proposed Approach + +1. Define sort buckets with priority: + - Priority 0: Overdue (due_date < today) + - Priority 1: Today with specific time (due_date == today && time != 00:00) + - Priority 2: Today all-day (due_date == today && time == 00:00) + - Priority 3: Future (due_date > today) +2. Within each bucket, sort by due_date ascending +3. Implement stable sort to preserve original order within equal items + +## Traps to Avoid +- **Timezone handling:** Ensure consistent timezone (user's local or UTC?) for date comparisons +- **Null due dates:** Decide where tasks with no due date should appear (suggest: after future tasks) +- **All-day detection:** Todoist may use different conventions; verify API response format + +## Affected Components +- `internal/handlers/handlers.go` +- `internal/handlers/handlers_test.go` + +## Definition of Done +- [ ] Tasks sorted in specified order +- [ ] Overdue tasks appear first +- [ ] Timed tasks today before all-day tasks today +- [ ] Future tasks appear last +- [ ] Null due date tasks handled gracefully +- [ ] Unit tests cover all buckets and edge cases |
