search

SherwynNg / pe

0 stars 0 forks source link

Unnecessary Capitalise #13

Open SherwynNg opened 4 months ago

SherwynNg commented 4 months ago

Steps to reproduce:

  1. "Edit 4 -t friend2"

Expected: I thought it will be able to read the command as an edit command and i thought all commands will be case-insensitive so "edit" is the same as "Edit"

Actual: It returns me an error "Invalid command"

Screenshot 2024-04-19 at 4.53.04 PM.png

nus-pe-script commented 4 months ago

Team's Response

Rejected - all commands are case-sensitive for standardisation.

The 'Original' Bug

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

Commands are identified with case sensitivity

For example, to open the help window, I must enter help instead of Help or HELP, which is too specific. This constraint should better be removed to improve user experience because it's quite possible that user accidentally enters variants of help.

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

Their Response to the 'Original' Bug

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

Currently, all commands are standardised case-sensitive to prevent ambiguity. However, this is a good suggestion for future versions of Dook.

Stated in UG:

Screenshot 2024-04-20 at 12.40.47 PM.png

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 response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** I feel that this is a feature flaw and should not be case-sensitive as there is no need and it slows down the typing of command.