search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Data loss + lack of error feedback when `contactRecords` is blank #246

Open nus-pe-bot opened 3 days ago

nus-pe-bot commented 3 days ago

Steps to reproduce

  1. Launch the app and save sample data by quitting properly.

  2. Edit the json file, delete all fields of contactRecords object and save.

  3. Start the app up.

image.png

While it is unlikely that humans will make this kind of error often, I think it's a severe issue because there is no error feedback— a confused user might run a command, which writes an empty list over the users corrupted data. Important data could be lost.

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

GabrielCWT commented 2 days ago

Team's Response

6185858435573203254.jpg

It is stated in our UG as well that there are no guarantees that the product/data will work if you change the data in the json file to make it invalid.

Duplicate status (if any):

--