search

yashpola / pe

0 stars 0 forks source link

Cannot move the addressbook.json file without erasing the data #6

Open yashpola opened 2 months ago

yashpola commented 2 months ago

Description

I wanted to change the location of the addressbook.json file but when I restarted the app after moving the file elsewhere, the app did not load with my data. The UG also does not specify how the application saves and retrieves my data each time so I made the assumption that it is able to track the file from anywhere

Expected Outcome

Since the behavior regarding moving the json file is unspecified in the UG, I would want the application to either be able to save my data from anywhere or warn me against changing the data file location.

Actual Outcome

My application before I quit the app:

image.png

My application after I changed the addressbook.json file's location and then restarted the application (data is gone):

image.png

nus-se-bot commented 2 months ago

Team's Response

Dear tester, we deeply want to thank you for your feedback.

severity.Low : A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only.

We would like to change the severity to low because the issue is rare to occur, as it is rare for users to move the json file.

Furthermore, as per our DeveloperGuide: https://ay2324s2-cs2103t-f14-1.github.io/tp/DeveloperGuide.html#8-11-saving-data it explains what happens when user deletes InternHub data file.

Screenshot 2024-04-20 at 3.25.32 PM.png

Additionally, as per our UserGuide: https://ay2324s2-cs2103t-f14-1.github.io/tp/UserGuide.html#4-4-1-saving-the-data

We did mention that the location of where the json is stored in. Perhaps we will also add to the UserGuide and not just the DeveloperGuide to make it more clear for users to not move the json file out of the data folder and expect the data to still be there.

Thank you

Yours Sincerely, JAYME

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]