JovenSoh / pe

0 stars 0 forks source link

Common Symbols used In Names Not Recognised #2

Open JovenSoh opened 1 week ago

JovenSoh 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.

nus-se-bot 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 /)

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]