search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Editing meeting times in JSON file to clash results in entire data being wiped on load #4684

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

Steps to Reproduce:

  1. Add two meetings with different times
  2. Edit the data file such that they have the same time
  3. Start up the application

Although you welcome users to try edit the file, small mistakes such as this (even if the timings overlap by 1 minute) will cause the entirety of the data to be wiped. A warning would be more appropriate.

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

Thing1Thing2 commented 2 years ago

Team's Response

Thank you for the bug report.

Referring to the section below given in the UG:

image.png

we believe that there was sufficient warning given to users on cautiously editing the data file.

Also, we feel that this issue correlates to the feature where the entire data file would be cleared on corruption. As suggested, it would be more appropriate for us to add a warning message. But, for now we would like to consider it as a feature flaw.

Duplicate status (if any):

--