ckcherry23 / pe

0 stars 0 forks source link

Can add all symbols as name #5

Open ckcherry23 opened 2 years ago

ckcherry23 commented 2 years ago

Steps to reproduce:

add n/#$(*($(# p/22224444 e/jamesho@example.com a/123,Clementi Rd, 1234665 c/01-01-2020 m/Avid hiker t/friend

Expected:

Error

Actual:

Accepted name

Screenshot(s):

Screenshot 2022-04-16 at 2.58.40 PM.png

nus-pe-script commented 2 years ago

Team's Response

In the 21st century, many unique names are surfacing, e.g. "X Æ A-12" or "Zeta-Jones". To be all encompassing and inclusive, we did not restrict special characters in the name. Restricting as you mentioned would be Overzealous input validation.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]