search

Tanziyanghenry / pe

0 stars 0 forks source link

Duplicate email #3

Open Tanziyanghenry opened 1 week ago

Tanziyanghenry commented 1 week ago

image.png

Expected: Error shown due to duplicate email as 2 people will not have the same email

Actual: Input is accepted

Suggestion: Might be better to use email as an identifier as well instead of only NRIC

nus-pe-bot commented 3 days ago

Team's Response

Thank you for pointing out that our app allows duplicate emails for separate people. However it is not mentioned in our user guide that duplicate emails are not allowed. Therefore, allowing duplicate emails should be allowed and that is the expected behaviour of the program.

Another point I hope you could consider is that young children who do not have an email address and whose parents wish to key in their emails for their children could do so. By allowing duplicate emails, parents can use their emails for their young children who do not have an email address.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]