feat: update task status handling and integrate into forms and component#127
feat: update task status handling and integrate into forms and component#127iamitprakash merged 1 commit intodevelopfrom
Conversation
…ents - Renamed TASK_STATUS_ENUM value from 'COMPLETED' to 'DONE' for consistency. - Added status field to CreateEditTodoForm, allowing users to select task status. - Updated EditTaskButton to handle status updates when editing tasks. - Enhanced TodoStatusTable to reflect the new status naming convention. - Integrated status handling in CreateTodoButton for task creation.
|
Caution Review failedThe pull request is closed. Summary by CodeRabbit
WalkthroughThe changes update the task status enum by renaming "COMPLETED" to "DONE" and propagate this update throughout the codebase. The task form schema and UI are extended to support an editable task status, and the task creation and editing flows are updated to handle the new status field. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant TaskForm
participant Backend
User->>TaskForm: Opens create/edit task form
TaskForm->>User: Shows fields including status (Todo, In Progress, Done)
User->>TaskForm: Fills form and selects status
TaskForm->>Backend: Submits task data with status
Backend-->>TaskForm: Confirms task creation/update
TaskForm-->>User: Updates UI with new status
Possibly related PRs
Suggested reviewers
Poem
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
components/edit-task-button.tsxOops! Something went wrong! :( ESLint: 9.30.1 ESLint couldn't find the plugin "eslint-plugin-react-hooks". (The package "eslint-plugin-react-hooks" was not found when loaded as a Node module from the directory "".) It's likely that the plugin isn't installed correctly. Try reinstalling by running the following: The plugin "eslint-plugin-react-hooks" was referenced from the config file in " » eslint-config-next/core-web-vitals » /node_modules/.pnpm/eslint-config-next@15.3.5_eslint-plugin-import-x@4.16.1_@typescript-eslint+utils@8.36.0_79fa352f41e97200dfc66b7389b5a599/node_modules/eslint-config-next/index.js". If you still can't figure out the problem, please see https://eslint.org/docs/latest/use/troubleshooting. api/tasks/tasks.enum.tsOops! Something went wrong! :( ESLint: 9.30.1 ESLint couldn't find the plugin "eslint-plugin-react-hooks". (The package "eslint-plugin-react-hooks" was not found when loaded as a Node module from the directory "".) It's likely that the plugin isn't installed correctly. Try reinstalling by running the following: The plugin "eslint-plugin-react-hooks" was referenced from the config file in " » eslint-config-next/core-web-vitals » /node_modules/.pnpm/eslint-config-next@15.3.5_eslint-plugin-import-x@4.16.1_@typescript-eslint+utils@8.36.0_79fa352f41e97200dfc66b7389b5a599/node_modules/eslint-config-next/index.js". If you still can't figure out the problem, please see https://eslint.org/docs/latest/use/troubleshooting. components/create-edit-todo-form.tsxOops! Something went wrong! :( ESLint: 9.30.1 ESLint couldn't find the plugin "eslint-plugin-react-hooks". (The package "eslint-plugin-react-hooks" was not found when loaded as a Node module from the directory "".) It's likely that the plugin isn't installed correctly. Try reinstalling by running the following: The plugin "eslint-plugin-react-hooks" was referenced from the config file in " » eslint-config-next/core-web-vitals » /node_modules/.pnpm/eslint-config-next@15.3.5_eslint-plugin-import-x@4.16.1_@typescript-eslint+utils@8.36.0_79fa352f41e97200dfc66b7389b5a599/node_modules/eslint-config-next/index.js". If you still can't figure out the problem, please see https://eslint.org/docs/latest/use/troubleshooting.
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (5)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![korbit-ai[bot]](https://avatars.githubusercontent.com/in/322216?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review by Korbit AI
Korbit automatically attempts to detect when you fix issues in new commits.
| Category | Issue | Status |
|---|---|---|
 |
Unnecessary Comment ▹ view |
Files scanned
| File Path | Reviewed |
|---|---|
| api/tasks/tasks.enum.ts | ✅ |
| components/todo-status-table.tsx | ✅ |
| modules/dashboard/components/create-todo-button.tsx | ✅ |
| components/edit-task-button.tsx | ✅ |
| components/create-edit-todo-form.tsx | ✅ |
Explore our documentation to understand the languages and file types we support and the files we ignore.
Check out our docs on how you can make Korbit work best for you and your team.
| /> | ||
| </div> | ||
|
|
||
| {/*status*/} |
There was a problem hiding this comment.
Unnecessary Comment 
Tell me more
What is the issue?
The comment 'status' adds no value and doesn't follow the capitalization pattern of other section comments.
Why this matters
Inconsistent or redundant comments can create noise and make code harder to scan.
Suggested change ∙ Feature Preview
Remove the comment entirely as the code is self-documenting through its field label.
Provide feedback to improve future suggestions
💬 Looking for more details? Reply to this comment to chat with Korbit.
2 participants
Date: 18 July 2025
Developer Name: @AnujChhikara
Issue Ticket Number
Description
Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
Screenshot 1
Test Coverage
Screenshot 1
Additional Notes
Note
I'm currently writing a description for your pull request. I should be done shortly (<1 minute). Please don't edit the description field until I'm finished, or we may overwrite each other. If I find nothing to write about, I'll delete this message.
Description by Korbit AI
What change is being made?
Refactor task status handling by changing the
COMPLETEDstatus toDONE, and integrate the task status enum into forms and components across the codebase.Why are these changes being made?
The refactoring is aimed at improving consistency and clarity in task status labels by renaming
COMPLETEDtoDONEto align it with common terminology used in task management contexts. Integrating the task status enum into forms and components ensures proper handling and validation of task statuses throughout the application.