search

seannzx / pe

0 stars 0 forks source link

Allowed to add contact who's names are all numbers #6

Open seannzx opened 3 years ago

seannzx commented 3 years ago

Can specify the contact's name to be solely numbers which is weird

nameisphone.PNG

nus-pe-bot commented 2 years ago

Team's Response

There might be names that contain numbers, e.g. Elon Musk's child. We have to include these cases too.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Yes I know there are names which contains numbers (which the code accepts and this is fine). This is the case where the name is JUST numbers entirely.

Also the Æ symbol in elon musk child's name is a special character which is not allowed by your code.

Adding of people with hyphen's in their name are also not allowed which can be considered an additional flaw after talking about this point