Skip to content

feat: add start field to task panel#283

Merged
its-me-abhishek merged 1 commit intoCCExtractor:mainfrom
Hell1213:feat/add-start-field-to-task-panel
Dec 21, 2025
Merged

feat: add start field to task panel#283
its-me-abhishek merged 1 commit intoCCExtractor:mainfrom
Hell1213:feat/add-start-field-to-task-panel

Conversation

@Hell1213
Copy link
Copy Markdown
Contributor

Added start date field support to task creation, allowing users to set when a task begins.

Checklist

  • Ran npx prettier --write . (for formatting)
  • Ran gofmt -w . (for Go backend)
  • Ran npm test (for JS/TS testing)
  • Added unit tests, if applicable
  • Verified all tests pass
  • Updated documentation, if needed

Additional Notes

The start field follows the same pattern as other optional date fields (due, wait, entry). Empty strings are used to indicate when no start date is set, consistent with existing codebase conventions.

Demo:

Screencast.from.2025-12-19.17-05-17.mp4

@github-actions
Copy link
Copy Markdown

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added DatePicker component for start date field in task creation

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added start parameter to addTaskToBackend function and conditional logic to include in request body

Copy link
Copy Markdown
Contributor Author

@Hell1213 Hell1213 Dec 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced empty string with actual start date "2025-03-01"

Copy link
Copy Markdown
Contributor Author

@Hell1213 Hell1213 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Completed self-review . Backend and frontend properly handle optional start field. Tests passing. Ready for review.

@Hell1213 Hell1213 force-pushed the feat/add-start-field-to-task-panel branch from c744b49 to 1285315 Compare December 20, 2025 07:04
@Hell1213
Copy link
Copy Markdown
Contributor Author

Hey @its-me-abhishek,
FYI before testing this PR, please set in date-picker.tsx.
This should allow PR #283 to be tested smoothly.

@its-me-abhishek
Copy link
Copy Markdown
Collaborator

Please checkout for conflicts introduced by the recent merge

@Hell1213
Copy link
Copy Markdown
Contributor Author

Please checkout for conflicts introduced by the recent merge

thanks, will resolve all conflicts and push shortly

@Hell1213 Hell1213 force-pushed the feat/add-start-field-to-task-panel branch from 1285315 to f233998 Compare December 21, 2025 08:55
@its-me-abhishek its-me-abhishek merged commit ce15591 into CCExtractor:main Dec 21, 2025
4 checks passed
ShivaGupta-14 pushed a commit to ShivaGupta-14/ccsync that referenced this pull request Dec 25, 2025
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.

2 participants