search

jiaxinnns / pe

0 stars 0 forks source link

addAppt error message only catches 1 error at a time #9

Open jiaxinnns opened 2 weeks ago

jiaxinnns commented 2 weeks ago

When inputting a command with multiple invalid parameters, only the first one will be flagged in the error message. This poses inconvenience to the user who has to type in the command all over again just to realise there was another error

the time and date are both invalid: image.png

nus-se-script commented 1 week ago

Team's Response

Functionality Bug -> Feature Flaw

Thank you for this issue. It is a good point to allow users to see all the errors at once instead of individually. We have implemented this for other commands (like addMedCon) but believed it is not necessary for addAppt but your argument is valid.

Items for the Tester to Verify

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]