nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

Users shouldn't be prevented from manipulating `savedata.json` #2545

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Constraint-Human-Editable-File:

But in the FAQ, it's clearly stated that savedata.json should not be edited by human.

So while the json file is a text file and with time and effort I can know how to manipulate it, I'd still say this somewhat violates the Constraint-Human-Editable-File constraint.


[original: nus-cs2113-AY2223S2/pe-interim#2320] [original labels: severity.Medium type.FeatureFlaw]

iantenkai commented 1 year ago

Team's Response

The report is correct in that the description in UG does not reflect the app's accommodation of project constraints in having the savefile be editable.

Therefore, this is a minor issue with the UG, in that the UG is editable, but doing so is advised against.

Duplicate status (if any):

--