search

swongts / pe

0 stars 0 forks source link

No warning thrown #18

Open swongts opened 3 years ago

swongts commented 3 years ago

image.png

Calorie goal is lower than calorie intake, but it was allowed

soc-se-bot commented 3 years ago

Team's Response

The user is allowed to exceed their calorie goal. If "calorie /remain" is executed, the user would see that FitNUS tells them that they have exceeded their calorie goal. A warning message would be nice to have but is not essential for the "calorie /set" command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Perhaps this could be changed to be a feature flaw at severity medium then. I don't think it is uncommon for users to set a calorie goal lower than their current intake.

From CS2113 website, "Similarly, if the feature is implemented to work in a certain way but it could have been implemented to work in a better way (from the end-user's point of view) without much additional effort, it should be accepted as a type.FeatureFlaw"

It does not really make sense to allow the user to set that goal without a warning message at least.