search

seannzx / pe

0 stars 0 forks source link

Error Message for upper case commands #1

Open seannzx opened 2 years ago

seannzx commented 2 years ago

Minor issue but what happens when the user accidentally types an Uppercase character in the commands, which is a common thing that happens when users type too fast

While it does not allow the command to proceed (which is correct) I think the error message can be more descriptive instead of being an "Unknown Command"

Toggle.PNG

nus-pe-script commented 2 years ago

Team's Response

That is a good suggestion, however this is not a bug as it does not hamper the user's usage of the application.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If the command has many fields, (i.e adding a person with 100 tags or a very long address) and everything is in the correct format but the user gets thrown this error, he/she might just spend quite awhile to look through the command to find the mistake just to realise its caused by an uppercase character in the command. This still technically causes an inconvenience.

Given that its a CLI, the case where a user accidentally types an uppercase character is common as well, especially if the inputs preceding it had uppercase characters (tags, address etc)