teojunda / pe

0 stars 0 forks source link

Command words are case-sensitive #2

Open teojunda opened 4 months ago

teojunda commented 4 months ago

Screenshot 2024-04-19 at 4.15.59 PM.png

LIST is not recognised as a valid command. This makes the app not optimised for fast typists. It would be better for command words to be case-insensitive.

soc-se-bot commented 4 months ago

Team's Response

All other parameters (such as "--name=" and "--email=") are case sensitive, and must all be in lower caps. We decided to be consistent in our command line convention.

Furthermore, typing in all lower caps instead of all upper caps OR mixed caps would save even more time as there is no need to press caps lock to switch capitalization at any point.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Referring to the course website (see below), making keywords case-sensitive when there is no need for it makes the command unnecessarily complicated. Some users may have the Caps Lock key on, or may accidentally capitalise some letters, causing the command to fail.

There is no need to enforce the consistency of lower case characters in the command words, since it does not add any value to the application or the user, but makes the command unnecessarily complicated.

Therefore, the strict constraints enforced on the command words cause inconvenience to some users and is not optimised for CLI.

Screenshot 2024-04-23 at 4.59.21 PM.png


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** This is not a cosmetic issue. Some users may have the Caps Lock key on, or may accidentally capitalise some letters, causing the commands to fail since they are case sensitive.