search

nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

JSON file permits loading of negative currency conversion values without any error #352

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Screenshot 2019-11-15 at 12.51.23 PM.png

[original: nus-cs2103-AY1920S1/pe-interim#354]

muhammadaljunied commented 4 years ago

Team's Response

It would not be logical for any user to input a negative currency conversion rate by altering the JSON values, we acknowledge that it is a bug IF altering the JSON manually was detailed to the user in the User Guide (which was not), however since the app does not allow the user to alter the conversion rates used within the app itself we consider this outside the scope of the app.

Duplicate status (if any):

--