search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Schedule restrictive naming validation #4426

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Details

We are not allowed to add schedules of the same title, however, it is common for us to have multiple same appointments at a different time, for e.g I visit the dermatologist every 3 months, but I can only have 1 Visit Dermatologist schedule for tomorrow and not in 3 months time because they are the same event

Steps to Reproduce

  1. schedule add h/Visit Dermatologist t/5/14/2024 0930 d/Discuss project details n/John Doe
  2. schedule add h/Visit Dermatologist t/9/14/2024 0930 d/Discuss project details n/John Doe (Error here)

[original: nus-cs2103-AY2324S2/pe-interim#3668] [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):

--