jiewei98 / pe

0 stars 0 forks source link

Unable to add contacts with same name #2

Open jiewei98 opened 2 years ago

jiewei98 commented 2 years ago

image.png

When trying to add a contact with the same name as an already existing contact into the app, the app rejects it and shows that "This contact already exists in the address book.".

Although this is stated in the UG that "Duplicate contacts with the same name, phone number, email or telegram handle cannot be added into the app.", it is normal for people to have the same name but different phone number, email, telegram handle.

I believe that this is a medium severity as it is a "flaw that causes occasional inconvenience to some users but they can continue to use the product.". Users will still be able to use the app but in the occasion that they meet two or more people with the exact same name, they will either not be able to save the contacts into the app or they would have to not use their full name when saving it (which does not solve the problem as you could save 1 contact as Alice and the other as Alice Lee but what if there was a 3rd Alice Lee?)

Hence, a possible solution would be to match the name field with either the phone number, email or telegram handle field as it is impossible for there to be more than 1 person with the same name and phone number, email or telegram handle.

soc-se-bot commented 2 years ago

Team's Response

We believe that the severity should be low as it is a bug that would only occur in very rare situations of contacts having the exact same given name and surname, and only cause minor inconvenience as users are able to simply add any further unique identifiers to the name to solve the issue (e.g. Alice Lee SoC).

It is a feature flaw instead of a functionality bug as it is the intended behaviour as stated in the UG.

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]