Skip to content

fix(focus): correct list query parameter names for focus ls#17

Merged
JeelyWu merged 1 commit into
masterfrom
fix/focus_ls
May 13, 2026
Merged

fix(focus): correct list query parameter names for focus ls#17
JeelyWu merged 1 commit into
masterfrom
fix/focus_ls

Conversation

@JeelyWu
Copy link
Copy Markdown
Owner

@JeelyWu JeelyWu commented May 13, 2026

Summary

Fix the query parameter names used in focus ls command from startDate/endDate to from/to to match the TickTick/Dida365 API requirements.

Changes

  • internal/ticktick/focus.go: Changed query parameters from startDate/endDate to from/to

Problem

When running tick focus ls against the Dida365 API, it would fail with:

Required String parameter 'from' is not present

The app layer already uses from/to naming in ListFocusInput, but the HTTP client was sending startDate/endDate to the API instead.

Test plan

  • Existing unit tests pass
  • Verified parameter names match API documentation

🤖 Generated with Claude Code

The TickTick/Dida365 API requires 'from' and 'to' as query
parameters for listing focus sessions, not 'startDate' and
'endDate'. This fixes the "Required String parameter 'from'
is not present" error when running `focus ls`.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@JeelyWu JeelyWu merged commit d980238 into master May 13, 2026
1 check passed
@JeelyWu JeelyWu deleted the fix/focus_ls branch May 16, 2026 14:24
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