search

ncmathan / pe

0 stars 0 forks source link

Invalid email allowed as input #5

Open ncmathan opened 10 months ago

ncmathan commented 10 months ago

image.png The UG states that the email has to be in the format username@domain.com but allows input "johnd@exampleom" as shown below

image.png

Steps to recreate: add n/John Doe p/98765432 e/johnd@exampleom g/MALE i/S2840182A b/02/01/1998 a/311, Clementi Ave 2, #02-25 pr/MEDIUM t/friends t/owesMoney

nus-pe-script commented 10 months ago

Team's Response

Hi, thanks for pointing this out. We believe that it is more of an issue with our documentation, as we intended for the program to accept valid emails (even without the '.com' appended at the back, but personal emails with a different domain format are rarer in our opinion, and they will be accepted by the program anyway, thus not impeding the usage of the app). However, this is reflected inaccurately in our User Guide. Apologies for the inconvenience and misunderstanding caused.

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]

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