search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Person names should not be case sensitive #1170

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

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.

Over here, edit the names of the person at index 1 and person at index 2 to 'Peter' and 'peter' respectively. However, they should be considered as duplicate persons.

Screenshot 2022-11-11 at 4.41.56 PM.png

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

yellow-294 commented 1 year ago

Team's Response

Added duplicate since both issues are considered similar under "duplication detection" as mentioned under course admin issue.

Duplicate status (if any):

Duplicate of #1432