Bug/232 Cannot add a new task on a selected date

[nikotj1]

closes #232

• [x] The pull request is complete according to the following criteria:
  • [x] Acceptance criteria have been met
  • [ ] The documentation is kept up-to-date
  • [x] Comprehensive tests (if applicable) have been generated and all pass.
  • [x] The pull request describes the changes that have been made, and enough information is present in the description for any developer to understand what has changed
  • [ ] Commits have been squashed (or will be on merge).
  • [ ] The branch name is descriptive and follows the pull request title format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
  • [ ] The pull request title is of the following format : {issue/bug...}/(Issue Number) - Name of issue. E.g bug/30-Fix-Project
  • [ ] The description uses github syntax to link to the issue. E,g Resolves se701g2/Doto#{Number}
  • [ ] At least two reviewers assigned. One of which must be the assigner of the issue.
  • [ ] If there are merge conflicts, run git rebase as opposed to git merge with master.

[jordansimsmith]

@PreetPatel could you have a look at this please, I believe you recently changed this logic.

[nikotj1]

LGTM, questionable about the comprehensive testing to be done, but if you can verify via a screenshot or something that'll be good! Nice work!

Thanks for the suggestion, screenshot attached:

[nikotj1]

I thought the idea of the task manager was to put in the tasks in the earliest coming up free time so that you could complete the task as early as possible. Correct me if I'm wrong, but right now this PR is looking like it gets added after the due date and time of the task - not particularly helpful if you have an asignment due then. Maybe it would be better to add in another input stating the start after date, specifying that you're only able to start the task after this date (so if I could only start it tomorrow, I'd specify tomorrow in the start after input and the task would then be made at that time

I genuinely misunderstood the application and perhaps the related issue could be improved, my bad.

I'm thinking of adding an extra field of earliest start time that would fulfill the same spirit of the related issue. I have created the new issue at #268