nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

UG does not explicitly specify that the imported CSV file will override the existing data in the app #5145

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


As per the title. I would like to consider this as quite a major bug as ALL data in the application will be lost without any prompt by the app, and I believe this import step is irreversible too.

As this is a HR management app, losing all the previous data without warning or a prompt could potentially be disastrous, as all employee data will be lost. I wanted to classify this as a feature flaw initially, since import is only useful if it 'adds on' instead of 'replace' for such a use case, however, I felt that you might have reasons for making a design decision to implement it as such. Thus I classified this under Documentation Bug instead.

Overall, I would propose that this is a medium severity bug either way.

Screenshot of the UG for reference:

image.png


[original: nus-cs2103-AY2122S1/pe-interim#5150] [original labels: severity.Medium type.DocumentationBug]

beefham commented 2 years ago

Team's Response

No details provided by team.

Duplicate status (if any):

--