search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Common Symbols used In Names Not Recognised #2978

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

Common names such as s/o indicating son of would not be accepted as a name although it is valid. This may restrict users with these in the name. Additionally, the error message is not as specific to this error.

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

IzN432 commented 1 week ago

Team's Response

image.png

There is no requirement for the full legal name to be inputted. This app is solely for the tutor's usage. No information is lost if they simply input s o instead of s/o for example. It would require extra work on our part to ensure that the name follows a specific format that includes s/o and other common formats, which takes lower priority over other implemented features. Hence, we argue that this is not in scope.

Additionally, the error message does state that only alphanumeric characters and spaces are allowed (which does not include /)

Duplicate status (if any):

--