LilyDaytoy / pe

0 stars 0 forks source link

Prompt message can be better #6

Open LilyDaytoy opened 2 years ago

LilyDaytoy commented 2 years ago

截屏2021-11-12 下午4.59.05.png

I forgot how the tag of the interval, and I typed "d/", so I got stuck in the this error and do not know how to do. Maybe better to include the "i/" in the message like Interval of item (month or year in case-insensitive format e.g., i/mOnTh, i/year, i/MONTH)

nus-pe-script commented 2 years ago

Team's Response

The error message already provides the information that Interval is missing/wrong, so it serves its purpose. As the user will already know that there is something wrong with Interval tag at this point, in the case the user is not sure what the tag was, he/she can always use the help command to refer back to the tag format: thus, it is not likely that user will get stuck at any time. Moreover, information about what tag to use is also stated in the User Guide. Changing to severity Low as it will only cause minor/rare inconvenience in the case where the user never refers to the help command or read the user guide.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

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

Reason for disagreement: [replace this with your reason]