diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-01-12 14:05:27 -1000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-01-12 14:05:27 -1000 |
| commit | 9ef5b7f37883f846f105da9dc5d2ba1415e594e3 (patch) | |
| tree | 7b39b9328c1b649e6bb87fba3ed6734448046afb /SESSION_STATE.md | |
| parent | e57671031d0e792926d12701aace4ffbff6a5bdf (diff) | |
Sort Trello boards with active boards first
Improves UX by prioritizing boards with cards over empty boards.
Both API and cached results now sort consistently: non-empty boards
appear first, then empty boards, with alphabetical ordering within
each group.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Diffstat (limited to 'SESSION_STATE.md')
| -rw-r--r-- | SESSION_STATE.md | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/SESSION_STATE.md b/SESSION_STATE.md index a663d91..55138c9 100644 --- a/SESSION_STATE.md +++ b/SESSION_STATE.md @@ -1,7 +1,7 @@ # Current Session State ## 🎯 Active Goal -Complete security test coverage for path traversal and SQL injection fixes. +Board sorting implementation complete. ## ✅ Completed - Initial Phase 1 feature set (Trello, Todoist, Obsidian, PlanToEat) @@ -25,6 +25,11 @@ Complete security test coverage for path traversal and SQL injection fixes. - internal/api/obsidian_test.go: TestGetNotes_SymlinkSecurity validates symlink protection - internal/store/sqlite_test.go: TestGetNotes_LimitClause validates LIMIT parameterization - 2 new test files with 7 total test cases, all passing + - **Commit:** e576710 "Add security tests for path traversal and SQL injection fixes" +- **UX Improvement:** Board sorting - non-empty boards first, then alphabetical + - internal/api/trello.go:220-228: Added sort logic to GetBoardsWithCards + - internal/store/sqlite.go:428-433: Updated SQL query to sort cached boards consistently + - Empty boards now pushed to bottom, active boards at top ## 🏗️ Architecture & Decisions - **Decision:** Use SQLite for caching with a 5-minute TTL. |