search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Schedule Command restricts Heading to be unique #822

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

image.png image.png Using the command schedule add h/House Visit t/2/14/2023 0930 d/Discuss project details n/Bernice Yu fails because the heading is the same as another existing event. This is a feature flaw as the person, time, and other parameters are all different and unique.

Heading should not be a unique identifier for an event as they are very common. If multiple people have the same common event for example "House Visit" or "Appointment" they will not be able to create an event. A suggested solution would be to use the Heading, Name and Date as an identifier instead

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

Rishit02 commented 4 months ago

Team's Response

Since this app is catered to social workers we see that it might be rare for them to have the same appointment headings. Moreover, since they are aware that unique headings are required they can create simple workarounds like adding a number to their appointment, eg: meet alex1 and meet alex2, therefore this seems like not a huge issue, thus we recommend reducing the severity to Low.

Duplicate status (if any):

Duplicate of #4426