soc-pe-bot
commented
1 year ago Team's Response
While there are constraints of requiring data to be stored locally and that they should be in a human editable text file, there are no constraints in the tp that require us to check for the integrity of the file that has been intentionally broken by a professional user (such as you).
The target audience of our application is made up of mostly non-tech savvy people, and the only people who would tamper with such data files are developers. With that in mind, we did not prioritize integrity checking as we'd rather focus on implementing the other parts of our application.
With that being said, it is a good function, but not necessary to have. Perhaps in future iterations of the product, this feature could be implemented.
Items for the Tester to Verify
:question: Issue response
Team chose [response.NotInScope]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
Describe the bug
The app will crash if the menu.json file is modified (refer to screenshots). In addition the error renders user unable to use other command the application as the .json file keeps throwing error.
Screenshots