nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Inconsistent Behaviours for new application Usage or corrupted data. #4258

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

When the data is corrupted, the app with start with an empty list. While if the data is starting for the first time, the app with started with a sample patient list. This inconsistency may lead to potential confusion to the users.

Screenshot 2023-04-14 150259.png


[original: nus-cs2103-AY2223S2/pe-interim#4012] [original labels: severity.Low type.FeatureFlaw]

valerietanhx commented 1 year ago

Team's Response

Hi!

Thank you for your input! This design was intentional.

Having sample data when the app is started for the first time is intended to give new users an idea of how to use the app and how it would look like when populated with data.

On the other hand, we do not see a need to load sample data if the user's data file is corrupted, since a user trying to import their own data is unlikely to need sample data as a guide. In fact, it might be misleading as it might appear that the user's data file has been loaded correctly, the incorrectness of the data only being noticed upon closer inspection. Moreover, starting with an empty list would prompt the user to check their data source.

Duplicate status (if any):

--