search

J3RN / time-tracker

A time tracking application
http://timesheet.j3rn.com
MIT License
6 stars 14 forks source link

Fixed due date bug on update #21

Closed J3RN closed 8 years ago

J3RN commented 8 years ago

Since the introduction of due dates, the JavaScript has always set the due date to be the current day's date, even if you are editing an older task whose due date is far in the future. This PR makes it so that dates will not be overridden and uses the pretty date picker pulled in for Time Entry start times.

UPDATED 2016-08-27 to reflect new approach.

J3RN commented 8 years ago

Instead of a JavaScript hack, the best way to attack this bug is probably to have the value of the checkbox sent to the controller, so the controller can toggle whether the value of the due date should be used or not.

J3RN commented 8 years ago

The added benefit of sending the checkbox is that you can also null out due dates after they have been sent, which does not currently work.

J3RN commented 8 years ago

Fixes #43

J3RN commented 8 years ago

Now, simply clearing the textbox will clear the due date.