Uh oh!
There was an error while loading. Please reload this page.
feat: add time to due date - #333
Conversation
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:
More information on how to conduct a self review: 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! |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
its-me-abhishek
left a comment
There was a problem hiding this comment.
This looks great overall, please look into these changes, will merge later once these get fixed
Description
This PR adds explicit date + time selection support for time-based Taskwarrior fields, starting with due.
Replaced DatePicker with a new DateTimePicker for the due field
Clear separation between date-only and date + time input
Time is included only when the user explicitly selects it
Backend now handles incoming date/datetime values to Taskwarrior format
Updated tests regarding the same
Fixes: Add additional time picker for time(and date) based fields in AddTaskDialog #325
Checklist
npx prettier --write .(for formatting)gofmt -w .(for Go backend)npm test(for JS/TS testing)Additional Notes