Errors on log when using checkmcs with string values of MC

[Jh123x]

Errors on log when using checkmcs with string values of MC

Steps to recreate:

1. Modify MC to non digit values
2. Key in checkmc

Command cannot be keyed and error appears on console

[nus-se-bot]

Team's Response

We agree that this should have been handled gracefully, however, as discussed here: https://github.com/nus-cs2103-AY2021S1/forum/issues/423, we have chosen to lower this to "low" for the following reasons:

• Only advanced users are expected to edit the JSON file, as such this should be a rare issue since most users are not "advanced"
• If a user is already "advanced" enough to edit the JSON file, it means that they wish to place a MC value of their choice. MC values are known to be numerical and no user would reasonably input alphabets. As stated in our UG:

• As mentioned in the forum post above, users can revert their change and still continue using the app normally

  Items for the Tester to Verify

  :question: Issue severity

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

• [x] I disagree

Reason for disagreement:

As per the image above, it does not fall into low category as it does not only cause a "minor inconvenience".

For this bug, the command doesnt work completely.

Based on the forum post that your team has provided, I believe it is of Medium severity. Instead of low.

---