search

avellinwong01 / pe

0 stars 0 forks source link

Unclear error message for delete schedule #10

Open avellinwong01 opened 2 years ago

avellinwong01 commented 2 years ago

image.png

For a user unfamiliar with the commands of the app, it is unclear what positive integer he is supposed to enter here.

nus-pe-bot commented 2 years ago

Team's Response

image.png

The format is specified as delete schedule INDEX since the only possibility for incorrect inputs (if the command is valid of course) would be INDEX. Since the input "exercise" is not a positive integer, the user would be able to assume that the only argument they input should be an positive integer. As such I do not see an issue with the error message.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Even if it can be deduced by the user, I believe it would still be better to make the error message more explicit that the integer should be for "index" without needing the user to infer (through reading the UG).Though I would agree that the severity is very low.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]