jonpwk / pe

0 stars 0 forks source link

Unable to Add Person with Valid Email #9

Open jonpwk opened 5 months ago

jonpwk commented 5 months ago

Command used: /add John Doe; 98765432; 1++1@example.com; Street A; 2024; Software Developer

From the User Guide, the constraints provided for "EMAIL" are: image.png

As per the command, the email provided 1++1@example.com contains @.

From the application itself, the constraints provided for "EMAIL" are:

1. 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.

As per the command, the email provided contains alphanumeric, 1, and the special character +.

nus-pe-script commented 5 months ago

Team's Response

1++1@example.com is not a valid email address. There are specific rules regarding characters, symbols, and format. One of them is that special characters cannot be consecutive. It's not feasible to list all the rules within the app due to length constraints. Moreover, if this email address cannot even be created in any email provider, there is no reason to store this in the app and it's logical to prevent its creation within our app.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If all the rules cannot be listed within the app due to length constraints, then it might be possible to list it in the User Guide as an alternative since the UG is used in tandem with the application.


## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]