nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Unable to add person after changing case of prefix #5155

Open nus-pe-bot opened 5 months ago

nus-pe-bot 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)


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

Joseph31416 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

Duplicate status (if any):

--