search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Property type parameter t/ is unnecessary when adding meetings #2132

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

Description

The current format for addmeeting includes t/TYPE:

addmeeting mt/MEETING_TITLE d/MEETING_DATE bp/BUYER_PHONE sp/SELLER_PHONE t/TYPE c/POSTAL_CODE

However, this is unnecessary since properties with different types but the same postal code cannot be added - so using postal code would be sufficient to uniquely identify the property location.

image.png

image.png

[original: nus-cs2103-AY2425S1/pe-interim#2929] [original labels: type.FeatureFlaw severity.Low]

jolwnn commented 1 week ago

Team's Response

No details provided by team.

Duplicate status (if any):

--