search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Case sensitivity for people management #4037

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

Command: Add, Edit contacts

Description: Unique contacts are taken with case sensitivity in consideration. This goes against the real world entity it represents "In general, case sensitivity of something should follow the case sensitivity of the real world entity it represents e.g., as person names are not case-sensitive in the real world, they shouldn't be case-sensitive in the app either."

Steps to Reproduce:

[Step 1] Add person with name "Sarah Woo"

[Step 2] Add person withh name "sarah woo"

Expected Behavior: Application to prevent duplicates of the same person from being added

Actual Behavior: Application allows users to add person with same name but different case for name

Screenshots/Logs:

image.png

[original: nus-cs2103-AY2324S1/pe-interim#3138] [original labels: type.FeatureFlaw severity.Medium]

cyr123456789 commented 7 months ago

Team's Response

We have mentioned this as a planned enhancement in our DG under Better way to handle duplicate contacts which means this should not be a feature yet in this iteration and hence we will be categorize this bug report as not in scope.

image.png

Duplicate status (if any):

--