nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Name field is case-sensitive #5730

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Expected: Suppose name "zanwen" is already in the data base, when adding another name "Zanwen", it should show an error message because "Zanwen" is already in the data base. Actual: It can add name "Zanwen" successfully, even though "zanwen" already exists.

image.png


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

zacchaeuschok commented 1 year ago

Team's Response

Rejected because we meant for the feature to work this way.

For whatever reason, users might sometimes want properties with duplicate names and we decided to give them a way to do that.

As an example, perhaps a user has an existing condo named ‘Admiralty’ (the property's name is 'Admiralty'), and he wants to quickly add a random unit thats in the Admiralty region. This feature thus allow him to add a new property with 'admiralty' as its name and an address specific to that unit.

Duplicate status (if any):

--