ling-ui / pe

0 stars 0 forks source link

Potential duplicate case not handled with warning #9

Open ling-ui opened 2 years ago

ling-ui commented 2 years ago

image.png These two entries of people have the same names with different capitalizations. This is not explained as a limitation in the UG and no warning is given which might cause users to add the same person twice

nus-pe-bot commented 1 year ago

Team's Response

Duplicate with #3289, both are regarding handling of duplicate entries in the database, in particular, handling capitalisation.

The 'Original' Bug

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

[Add] Duplicate Residents

Steps to reproduce: Command 1: add n/John doe j/Test d/Administration Command 2: add n/John dOe j/Test d/Administration

Expected: Coydir deems the same name with different capitalizations as two different employees.

While it is entirely possible that two employees have the same name, and same role, and same department. Coydir should at least warn the user of such a circumstance, and allow the user the freedom to proceed or not.

Refer to the images below for their name, position, and department.

image.png

image.png


[original: nus-cs2103-AY2223S1/pe-interim#2875] [original labels: type.FunctionalityBug severity.Low]

Their Response to the 'Original' Bug

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

Hi there! Thanks so much for the report.

We acknowledge this bug. However, we did not to intend handle duplicates in a manner like what you suggested, thus we feel that issue is more of a Feature Flaw rather than a functionality bug.

Once again, thank you so much for informing us regarding the error.

Cheers, Coydir HR is hard, Coydir is easy (est. 2022)

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 severity

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

Reason for disagreement: [replace this with your explanation]