jialatteo / pe

0 stars 0 forks source link

31 February is allowed as a meeting date #8

Open jialatteo opened 1 year ago

jialatteo commented 1 year ago

addM l/007 n/Joke Peralta d/2022-02-31 12:00 is allowed even though 31 February does not exist.

Success message after executing the command is shown below:

image.png

nus-se-script commented 1 year ago

Team's Response

A severity of low seems more appropriate here as a user inputting a date of 30th February is unlikely, and if done so does not affect usage of the product greatly.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: The application should at least give a warning message or block this command.
The application also does not check for meeting clashes.

In the scenario that: