search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

[Add] Duplicate Residents #3289

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

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]

kevinchangjk commented 1 year ago

Team's Response

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)

Duplicate status (if any):

--