wx-03 / pe

0 stars 0 forks source link

NaN accepted for weight and height #8

Open wx-03 opened 2 weeks ago

wx-03 commented 2 weeks ago

image.png

image.png

Steps to reproduce:

  1. Run update userdata
  2. Enter NaN when prompted to input height and weight

Expected: Input is rejected, user is prompted to enter again

Actual: Input is accepted and height and weight are saved as NaN

nus-pe-bot commented 1 week ago

Team's Response

We wanted to give the user the option not to share very sensible data with us since the key feature, i.e. tracking calorie consumption is still usable without sharing these personal details with us. In such a case, however, the daily target calorie consumption cannot be determined and is set to 0. As an intermediate compromise for the current stage, we did not disable the features that depend on a reasonable target calorie number (the consumption bars etc.). Therefore, the app remains usable while the user experience is arguably reduced because the consumption bar visuals are created with respect to a target consumption of 0.

Therefore we accept this as a bug, but reprioritize it to Very Low.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]