Skip to content

fix: use /tasks for getAllTasks on Vikunja 2.x (fixes #40) - #89

Open
joyjit wants to merge 1 commit into
democratize-technology:mainfrom
joyjit:resubmit/fix-get-all-tasks-endpoint
Open

fix: use /tasks for getAllTasks on Vikunja 2.x (fixes #40)#89
joyjit wants to merge 1 commit into
democratize-technology:mainfrom
joyjit:resubmit/fix-get-all-tasks-endpoint

Conversation

@joyjit

@joyjit joyjit commented Jul 26, 2026

Copy link
Copy Markdown

Summary

  • Fixes [BUG] #40: all-project task lists fail on Vikunja 2.x because node-vikunja 0.4.0 still calls GET /tasks/all.
  • After each client is created, rebind getAllTasks to GET /tasks (the path Vikunja 2.x serves).
  • Temporary shim until node-vikunja#4 ships and this package can bump the dependency.

Test plan

  • Unit tests for the path shim
  • Factory test that patched getAllTasks hits /tasks
  • CI green on this PR
  • Confirm unscoped / all-projects task list works against Vikunja 2.3

@joyjit
fix: use /tasks for getAllTasks on Vikunja 2.x
a0a32cd 
node-vikunja 0.4.0 still calls /tasks/all, which Vikunja 2.x
rejects. Rebind getAllTasks after client creation so all-project
listings work until node-vikunja publishes a fix.
@joyjit joyjit mentioned this pull request Aug 5, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG]

1 participant

@joyjit