search

tanyizhe / pe

0 stars 0 forks source link

Save file not in human readable text file. #5

Open tanyizhe opened 1 year ago

tanyizhe commented 1 year ago

image.png

image.png

Save data is not stored in a text file. This violates the "Human Editable File" constraint of the tp.

nus-pe-script commented 1 year ago

Team's Response

It is a json file, a form of text file.

Json files are also human readable and editable

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The json file is read-only. This complicates testing for any tester, and given the time constraints during the PE, I think that this does not respect the constraint of the tP.

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** It is not just a Feature Flaw, it is functionally more difficult for me to edit the file having to go through json and the read-only part.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** This issue is not just cosmetic, it directly affects functionality.