nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Duplicate reminders allowed under the reminder list #398

Open nus-pe-bot opened 6 days ago

nus-pe-bot commented 6 days ago

Context : 2024-01-01 12:00 Appointment for John Doe already exists in the current list of reminders

Inputted command : ra n/John dt/2024-01-01 12:00 d/appointment

Expected behaviour : Unable to add this reminder for John Doe as it is a duplicate reminder.

Actual behaviour : A new reminder section is added for John Doe, even though it is the same thing.

Suggested solution : Make the description for reminders case insensitive so as to avoid duplicate reminders from being added into the reminder list. This can prevent the overcrowding of the reminder list if users accidentally input duplicate reminders for the same person.

Screenshot 2024-11-15 at 5.23.31 PM.png


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

jereeemyyyy commented 5 days ago

Team's Response

Duplicate status (if any):

--