search

Johnson-Yee / pe

0 stars 0 forks source link

Corruption of config file #14

Open Johnson-Yee opened 3 years ago

Johnson-Yee commented 3 years ago

Issue description

Steps to reproduce the issue

  1. Edit the config.txt file
  2. remove e from true

image.png

What’s the expected result? Corrupted data detected. Re-initalising data

What’s the actual result? image.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for bringing this up. Unfortunately, we were using the Boolean.parseBoolean function, where anything that is not "true" is returned as false. This results in the application thinking that it is on its first run, hence, the prompt as such. We downgraded this from Medium to Low as the rest of the functionalities of the application still works fine. Only the name and recommended hours are resetted as a result of this.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]