search

albertsutz / pe

0 stars 0 forks source link

using s/o for name is invalid #5

Open albertsutz opened 2 years ago

albertsutz commented 2 years ago

image.png

The reason why severity is Medium and not low is because "s/o" is a very common in names, especially in Singapore Culture

image.png

soc-pe-bot commented 2 years ago

Team's Response

From CS2103T Website: https://nus-cs2103-ay2122s2.github.io/website/admin/tp-pe.html#:~:text=Use%20of%20symbols,of%20the%20person.

Use of symbols in input values: It is acceptable to disallow certain characters in input values if there is a justification (e.g., because using those symbols in an input value makes the command harder to parse), but they can still be considered FeatureFlaw bugs if they cause inconvenience to the user. For example, 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.

The names input into the application are not expected to match the legal name of the person. They are just supposed to be a name that is recognisable by the user of the application.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]