tengcharmaine / pe

0 stars 0 forks source link

Unable to add person after changing case of prefix #5

Open tengcharmaine opened 5 months ago

tengcharmaine commented 5 months ago

Screenshot 2024-04-19 at 4.30.50 PM.png

What I did: According to the screenshot above, I changed the prefix of the phone attribute (p) to upper case and was not able to add the contact.

Maybe yall could have made it such that all the prefixes accept upper and lower cases. Such a thing could happen to users if they were to be typing fast and ending up with a typo (in this case an accidental capitalisation)

nus-pe-script commented 5 months ago

Team's Response

Thank you for the suggestion! We feel that having non-case sensitive command delimiters is less useful work than has been done so far as it is easily correctable by the user (just a one letter change) and in addition, a suitable error message is shown which clearly tells the user what the correct command format is. Hence, it is likely NotInScope but we will consider allowing non-case sensitive command delimiters in the future.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]