search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Bugs on renaming the files #4111

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months ago

I think the original file name netconnect.json is not cool so I changed the file name of the data source to xxx.json, when I open the app again, I didn’t realized this is a brand new contact list , so I keep editing on this new address book, after several commands, I found that this is not the contact list before. After certain thinking, I intuitively go back to the source directory, delete the new netconnect.json and rename my xxx.json address book back to netconnect.json … and then bad thing happens, I got my old contact list overridden by the new contact list. I lost all my data, it made me lose all my business partners just because I changed a file name… I think it is better to add a change data source command to solve this problem, or you can modify the storage mechanism.

[original: nus-cs2103-AY2324S2/pe-interim#5167] [original labels: severity.Medium type.FeatureFlaw]

shaunlxw commented 6 months ago

Team's Response

image.png

Under "Editing the data file" section in the UG, it specifically said that data is saved in netconnect.json, i.e. data in any other file (e.g. "xxx.json") is not used as the application's data file. Additionally, there is a caution in the same section warning about unexpected behaviour from invalid editing of the data file.

Also seems like you are suggesting an separate and additional feature to change the data source, which isn't related to the features delivered in v1.4.

Duplicate status (if any):

--