search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Misinformation in the UG #1733

Open nus-pe-bot opened 2 months ago

nus-pe-bot commented 2 months 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.

image.png

image.png

Not all the data is deleted when the data file is manually edited. Here, I edited the category of the 1st person to be 'Name' which should not be allowed since it already exists. However, only the data for the 1st person is deleted.

[original: nus-cs2103-AY2324S2/pe-interim#1424] [original labels: severity.Low type.DocumentationBug]

Cedricaca commented 2 months ago

Team's Response

Works exactly as seen. Certain edits to the addressbook will cause our app to work in unexpected ways. In this case, it was editted incorrectly by adding in a duplicate category when duplicate categories are explicitly not allowed in our app, causing it to work in an unexpected way, in this case deleting all entries of that person. Looks like a user error rather than a bug, who ignored the warning message to not edit the json file unless confident to be correct.

Duplicate status (if any):

--