Nauxe / pe

0 stars 0 forks source link

Vague error message for `at` command with invalid type #5

Open Nauxe opened 12 months ago

Nauxe commented 12 months ago

Error given does not match expectations of the user. The user is given the error message telling them that the command format is incorrect, but may be instead due to invalid inputs being put into the parameters instead.

image.png

This is medium severity due to users commonly mistyping due to fast typing speeds and possibly entering in a wrong input wondering what is wrong.

nus-se-script commented 12 months ago

Team's Response

Thanks for your response, however, the error message is working as intended due to the presence of an index in your input command.

As mentioned in our UG, users were advised to read the section on "Command Breakdown".

Screenshot 2023-11-19 at 3.39.55 PM.png

Through the command breakdown section, we had explicitly explained the difference in (ARGUMENT) and (PREFIXES). Screenshot 2023-11-19 at 3.40.47 PM.png

Therefore, it is reasonable to expect users to follow the command instruction by add with no index, as by providing an index, it is not complying to the expected command format. Thus, this falls under an invalid command format.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]