nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Wrong value in json file not caught #827

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Loading from json file should have thrown "Error reading file". Steps:

image.png


[original: nus-cs2113-AY2122S1/pe-interim#802] [original labels: severity.High type.FunctionalityBug]

jonathanmui4 commented 2 years ago

Team's Response

Granted that this is a bug and that we should have converted the String isDone into lower case before loading the data from the json file. However, I downgraded the severity to low because:

Thus, this bug does not affect most users and does not cause major problems for users.

Duplicate status (if any):

--