nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Changing json file #3237

Open nus-se-script opened 5 days ago

nus-se-script commented 5 days ago

I changed the json file to have incorrect inputs like name having symbols when suppose to be alphanumeric but the app still displays the old one this goes against the UG and DG


[original: nus-cs2103-AY2425S1/pe-interim#2058] [original labels: severity.Medium type.FunctionalityBug]

tayxuenye commented 4 days ago

Team's Response

Thank you for your bug report. We acknowledge that PROperty does not have any checks for invalid JSON edits. However, we have also stated explicitly in the User Guide, as seen in the screenshot below, that modifying the data file with edits that are invalid may cause PROperty to behave unexpectedly, and we strongly discourage this.

We have also downgraded the severity to Low since most users will not be editing the JSON file at all under normal usage. It is much easier to use the application to edit names than accessing the JSON file.

Screenshot 2024-11-18 140225.png

Duplicate status (if any):

--