search

avellinwong01 / pe

0 stars 0 forks source link

Possible improvement to error message for list schedule command #11

Open avellinwong01 opened 2 years ago

avellinwong01 commented 2 years ago

image.png

Since list schedule all is also a valid input, perhaps it should be mentioned in the error message, rather than just the date and time

nus-pe-script commented 2 years ago

Team's Response

Valid point, accepted as special keyword "all" is also valid in addition to date in the format dd/mm/yyyy and the error message could be more inclusive of this fact.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: I don't think this is a purely cosmetic bug as the error message can possibly mislead users into forgetting that list schedule all is a valid command.