Editting JSON file enables your number to be above 99 and does not cause an error.

[alfredkohhh]

[nus-pe-script]

Team's Response

We allowed for this to happen because we wanted to allow advanced users such as more popular home bakers to be able to edit orders which are more than 99 but did not want to allow less advanced users from accidentally inputting a wrong value (above 99) into the app directly. This is because our app is targetted for home bakers and having orders larger than 99 items is very unlikely. Thus, we only allowed this to happen for advanced users.
We also specified that editing the json file is meant for advanced users only.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

• I disagree

Reason for disagreement: The team stated that this is "allowed to happen" but however, the functionality of editing any value above 99 will cause an error. This shows that the team did not handle cases whereby an advance user can directly edit the JSON file.

For Example, if an advanced user would want to use the edit order command (edito), as shown above, then it does not make sense for advance users to also edit other JSON fields, which can potentially mess up your data.

---

❓ Issue severity

Team chose [severity.Low]
Originally [severity.Medium]

• I disagree

Reason for disagreement: In accordance with all applications, when new users use your product long enough, the majority of them should and would become "Advanced users". It does not seem that the team had taken care of JSON edits with values that exceed their validation.

---