nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Consecutive special characters breaks email validation #1578

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

e.g. add n/John Doe p/98765432 e/j..o.h_n+d@example.com c/SG t/friends t/owesMoney

According to ug:

The local-part should only contain alphanumeric characters and these special characters, excluding the parentheses, (+_.-). The local-part may not start or end with any special characters

This example command seems to fit the pattern, however it cannot be added


[original: nus-cs2103-AY2324S2/pe-interim#1995] [original labels: type.FunctionalityBug severity.High]

chiralcentre commented 4 months ago

Team's Response

Thank you for the feedback. We acknowledge that this is a valid bug, but we would like to highlight that not accepting consecutive special characters is intended. From an online source, it can be seen that the local-part of email addresses cannot have 2 consecutive special characters. Hence, we believe the severity of this bug should be low because it is a flaw that is unlikely to affect normal operations of the product. Job applicants will not have email addresses that contain 2 consecutive special characters.

image.png

severity_labels.jpg

Duplicate status (if any):

--