search

yespiggy / pe

0 stars 0 forks source link

Missing Explanation for Possible Duplicate Contacts #6

Open yespiggy opened 2 months ago

yespiggy commented 2 months ago

image.png

I realized that after I did an edit command, there is one line that is not explained in the UG. Possible duplicate contacts: JohnDoe It seems that the application is able to detect potential duplicates. The UG does not explain how this being achieved.

If this is intended as a way to remind users of any potential duplicates, it should also list Johnoe who has the same phone number and email as JohnDoe, both of which are potential duplicates. Similarly, JohnnDoe is also a potential duplicate, but it is not detected either. In this case, a legitimate user behavior might not be well considered enough.

image.png

nus-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Unsure how duplicate detection works

Screenshot 2024-04-19 at 5.07.25 PM.png Insufficient details on how duplicate detection works. Only mentions that when similar name detected, users will be warned. However there was no such detection.

Screenshot 2024-04-19 at 5.08.54 PM.png

"Possible duplicate contacts:" only shows up when editing anything but name Screenshot 2024-04-19 at 5.09.05 PM.png

[original: nus-cs2103-AY2324S2/pe-interim#1071] [original labels: severity.Medium type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Agreed that this is an issue. Low because most users would not have to look into the specifics of near-match duplicate detection, given the app still allows users to add near-match duplicates. Additionally, this situation is not common.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :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.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]