emzm2023 / pe

0 stars 0 forks source link

Command names case-sensitive #8

Open emzm2023 opened 1 year ago

emzm2023 commented 1 year ago

Currently commands are case sensitive, ie.: "Customer edit" will throw an error. Might be good to make them case-insensitive since likely that users might not consider this to be an issue while using the app, so they will likely be confused as to why the command is not working.

nus-pe-bot commented 1 year ago

Team's Response

The UG has stated that command names and prefixes are case-sensitive. However, this is not as important because as fast-typist, it is unlikely that they would capitalise their commands as that would only add more time. Thus, this is NotInScope as it is specified in the UG and rectifying it is less important than the work that has already been done as it's not as easy as it seems as per the definition of NotInScope.

Screenshot 2023-11-20 at 12.13.28 AM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]