LiuFangrui / pe

0 stars 0 forks source link

Confusing clear command description in User guide #9

Open LiuFangrui opened 1 year ago

LiuFangrui commented 1 year ago

Clear command description: "Clears all stored data (excluding gym locations) on all days from NutriGoals. " As a user, I did not expect gym locations to be cleared. Specifying that gym locations are not cleared suggests to me that other app data not input by me are cleared, such as the default calories for find, e.g find rice will not longer show me "Calorie content of rice: 260 calories". It seems unnecessary to specify that gym locations are cleared as users cannot store gym data to begin with (it comes with the app)

nus-pe-script commented 1 year ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

The json file stores our gym locations, hence it could have had the potential to be cleared. On the other hand, the default calories are not stored in the json file at all, hence they do not have the potential to be cleared. Furthermore, we did this as we have plans to allow users to input other gym locations in the future and was future proofing our description.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]