Skip to content

Kashish task api - #70

Open
krutzia wants to merge 2 commits into
rohit-ups:mainfrom
krutzia:kashish-task-api
Open

Kashish task api#70
krutzia wants to merge 2 commits into
rohit-ups:mainfrom
krutzia:kashish-task-api

Conversation

@krutzia

@krutzia krutzia commented Aug 15, 2026

Copy link
Copy Markdown

Summary

  • Added unit tests for taskService
  • Added integration tests for the task API routes
  • Fixed the pagination offset bug
  • Fixed the task priority bug when completing a task
  • Implemented PATCH /tasks/:id/assign
  • Added assignee validation and whitespace trimming
  • Added support for reassigning an already assigned task
  • Added BUGS.md and implementation notes

Bugs Fixed

  1. Pagination now uses (page - 1) * limit so page 1 correctly returns the first tasks.
  2. Completing a task now preserves its existing priority.

Testing

  • 63 tests passing
  • Coverage: 96.12% statements
  • Coverage: 90.69% branches
  • Coverage: 93.33% functions
  • Coverage: 95.91% lines

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