search

hsiaojietng / pe

0 stars 0 forks source link

Does not allow some characters for name #3

Open hsiaojietng opened 2 years ago

hsiaojietng commented 2 years ago

How to reproduce: 1) Input edit 2 n/Saravan s/o Ravindraran or edit 2 n/Saravan s-o Ravindraran

Expected result: The app should allow the editing/adding of name that consists of 's/o' (represents son of) since it is a valid name.

Actual result: The app displays an error message as shown below.

image.png

soc-se-bot commented 2 years ago

Team's Response

We recognise that "S/O", "A/L", "D/O" and "A/P" are valid names. Unfortunately to the latest milestone, our product does not support the use of non-alphanumeric characters in a name, as stated in the error message.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: As stated in the PE section in CS2103 website, there is a need to mention and justify the reason for not including these valid names in the UG, which is not done so.

image.png