nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Overzealous Name field #3496

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

In actual world usage, there are people with / in their names like certain Indian names. But the Name field is overzealous and does not allow for people with non-alphanumeric characters.

Steps: 1) Type add n/Joh s/o pritam p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/3.50/4.00 g/male u/Nanyang Polytechnic gd/05-2024 m/Computer Science ji/173296 jt/Software Engineer Intern t/rejected t/KIV into command box

Expectation: Create a user with name Joh s/o pritam Actual: Show error Names should only contain alphanumeric characters and spaces, and it should not be blank

Screenshot 2022-11-11 at 4.26.47 PM.png


[original: nus-cs2103-AY2223S1/pe-interim#3176] [original labels: type.FeatureFlaw severity.Medium]

maryjess commented 1 year ago

Team's Response

We set the severity to Low because in reality it is very rare for names to contain symbols such as /

image.png

Duplicate status (if any):

--