search

cshao02 / pe

0 stars 0 forks source link

Use of "/" not allowed for name field #3

Open cshao02 opened 2 weeks ago

cshao02 commented 2 weeks ago

image.png

Users are not allowed to enter "/" for name field even though it is possible for names to use "/". Disallowing s/o in a person name because / is used as a command delimiter can cause a major problem if the input is expected to match the legal name of the person. Therefore, it would be better if "/" was allowed

nus-se-bot commented 1 week ago

Team's Response

The application was not designed with the legal names of each resident being used by the dorm manager in mind, given that it is unlikely that records from the app will be used for legal purposes. Specifically, we expected the dorm manager to use this to informally keep track of their residents, and expected them to name the students however they wanted in the application.

Accordingly, we allowed characters like commas, apostrophes, dashes and even numbers to make it easier for the user to format the names but did not implement inclusion of the slash character to prevent errors that may arise when parsing the command and its parameters.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]