search

tsx0314 / pe

0 stars 0 forks source link

Missing date checker #5

Open tsx0314 opened 1 year ago

tsx0314 commented 1 year ago

For deadline features, if the date has been passed. The application does not show error message

image.png

soc-se-bot commented 1 year ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Same issue as #869, the user should be able to freely add valid dates whenever he/she wants. It is not a missing date checker.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Deadline accepts past year (Date from the past)

image.png

Given deadline is for future events, dates from past years can be added without restriction/warning.

Steps To Reproduce

/deadline/add asdjadmsakdmkassda /by 30-12-2012

Expected

Prompt or restriction that prevent user from adding deadlines that is from the past (compared to current time)

Actual

User can add deadlines which have past

[original: nus-cs2113-AY2223S2/pe-interim#684] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

While we accept that there could be a reminder to the user when adding past dates, it could also serve as a reminder for the user that there is an overdue task which has yet to be completed. The deadline list serves like a task todo list where users can freely add what task they want and when they want the due date to be.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your reason]

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]