4llysa / pe

0 stars 0 forks source link

Case sensitivity of command #4

Open 4llysa opened 2 months ago

4llysa commented 2 months ago

Currently, the commands are case-sensitive even though there is no need for them to be.

Input: ADD n/Test Name m/A1234567Z e/johnd@u.nus.edu tl/johndoe01 t/cs2109s t/G19Group1

Expected output: When i use "ADD" instead of "add", I expect the app to either recognise it as "add" anyway, or remind me that "add" should be in small letters.

Actual output:

image.png

soc-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Wrong error message for upper case commands

Screenshot 2024-04-19 at 4.14.56 PM.png

Commands should be case in-sensitive and this may bring about inconvenience to the users if they accidentally type the command in upper case but the error message shows an unknown command.


[original: nus-cs2103-AY2324S2/pe-interim#5098] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

UG should have clearly mentioned the case-sensitivity of all command words.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [x] I disagree **Reason for disagreement:** Please refer to the guidelines of what is considered a FeatureFlaw. ![image.png](https://raw.githubusercontent.com/4llysa/pe/main/files/a4177bca-89ea-4793-a280-0f6f7b34c14d.png) It has been explicitly stated that the unnecessary case-sensitivity is a FeatureFlaw as there is no need for the commands to be in upper case. Hence, this is a FeatureFlaw and not related to your documentation.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** I quote from the guidelines for severity: severity.VeryLow : A flaw that is purely cosmetic and does not affect usage e.g., a typo/spacing/layout/color/font issues in the docs or the UI that doesn't affect usage. Only cosmetic problems should have this label. This FeatureFlaw is not purely cosmetic. It affects the user's precise usage of only lowercase letters which requires the user to be extra careful of ensuring that when dealing with every command. This could even be severity Medium because it is a constant inconvenience to every command.