nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Schedule add command doesn't allow for duplicated schedule names #4448

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Description: as stated in topic.

Steps to reproduce: 1.schedule add h/aaa t/12/01/2022 0223 d/here n/Gerald 2.schedule add h/aaa t/2/2/2024 0800 d/not here n/Irene

Expected behaviour: Command should be executed as the same schedule title can be used for multiple events

Actual behaviour: Command fails even though the schedules are with different people and at different times.

Screenshot 2024-04-19 at 5.08.33 PM.png


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

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