thikhinab / pe

0 stars 0 forks source link

Email is not case insensitive #4

Open thikhinab opened 2 years ago

thikhinab commented 2 years ago

Command: addc n/joseph p/89993234 e/joseph@nus.edu.sg u/test t/Colleague Lecturer

Add person with same email with different case

Before:

image.png

After:

image.png

Expected: The person should not be added.

nus-pe-bot commented 2 years ago

Team's Response

Does not contradict expected response. We do not block duplicate emails.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I agree that the severity should not be high, since it does not make the application unusable. However, considering the considering the Guidelines for Bug triaging under PE we have to consider case sensitivity as feature flaw. Furthermore, the email is real life entity which cannot be assigned to two contacts (given the case changes). According to your application it would be mean, two or more people can have the same personal email address which would not be the case in real life.

Screenshot:

image.png


:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]