nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Duplicate checking is case-insensitive #5647

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

[How to reproduce]

Run the command add n/Johnson Doe p/98765432 e/johnson@gmail.com a/2 Sengkang Avenue c/1A t/Student Council

Run the command add n/johnson doe p/98765432 e/johnson@gmail.com a/2 Sengkang Avenue c/1A t/Student Council

[Expected]

Duplicate to be detected

[Actual]

image.png

The second johnson doe is added although they are the same but missing capitalisation. It makes sense to detect these as duplicates given all their details are the same


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

jgyj123 commented 1 year ago

Team's Response

No details provided by team.

Duplicate status (if any):

--