Closed Bug 1032343 Opened 10 years ago Closed 10 years ago

Able to create task with end date prior to start date

Categories

(Mozilla QA Graveyard :: One and Done, defect)

Version 2
x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Milestone 2

People

(Reporter: rbillings, Assigned: impossibus)

References

()

Details

Attachments

(1 file)

Users are able to create tasks that end before they begin. There should be an error when selecting the date or after clicking Add Task.
Assignee: nobody → mjzffr
Comment on attachment 8451014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151

Goo start, Maja. Please address the comments in the PR and re-flag me for review.
Attachment #8451014 - Flags: review?(bob.silverberg) → review-
Attachment #8451014 - Attachment description: Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151 → Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151/files
Attachment #8451014 - Flags: review- → review?(bob.silverberg)
Comment on attachment 8451014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151

Looks good, Maja. Please just add a unit test (or two) and it should be ready to merge.
Attachment #8451014 - Flags: review?(bob.silverberg) → review-
Comment on attachment 8451014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151

I added some tentative tests to the PR and would like to check if I'm on the right track. (p.s. Not sure if I should be requesting review or feedback, Bob -- does it matter for your work flow?)
Attachment #8451014 - Attachment description: Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151/files → Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151
Attachment #8451014 - Flags: feedback?(bob.silverberg)
Comment on attachment 8451014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151

Looks good. Proceed with adding another test and refactoring the duplicate code. Thanks Maja.
Attachment #8451014 - Flags: feedback?(bob.silverberg) → feedback+
Attachment #8451014 - Flags: review- → review?(bob.silverberg)
Comment on attachment 8451014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151

Just one teeny tiny change left.
Attachment #8451014 - Flags: review?(bob.silverberg) → review-
Comment on attachment 8451014 [details] [review]
Link to Github pull-request: https://github.com/mozilla/oneanddone/pull/151

Thanks, Bob. :) Teeny tiny change made, commits squashed.
Attachment #8451014 - Flags: review- → review?(bob.silverberg)
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Attachment #8451014 - Flags: review?(mozpankaj1994)
Verified on staging
Status: RESOLVED → VERIFIED
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: