nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Medication Still Modified With Invalid Input #1217

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

When modifying the first index on the medication list (1 which is Panadol), into "21" which is invalid. I got thrown with the following error

Screenshot 2024-04-19 165143.png

When looking at the list of medications, using "list -t all" entry number 1 does not change. However when using list -t today. The first entry name becomes modified into "21"

Screenshot 2024-04-19 165439.png


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

royaliceblue commented 4 months ago

Team's Response

Thank you for reporting this. I acknowledge the presence of this bug. This is due to the rollback process of the Medication name that did not account for the fact that DailyMedication name has been changed, which eventually resulted in this bug occurring.

Duplicate status (if any):

--