search

ryan-txn / pe

0 stars 0 forks source link

list-food command does not list correctly #14

Open ryan-txn opened 1 week ago

ryan-txn commented 1 week ago

image.png

save file that caused the issue provided below as I suspect it might have caused the inability to detect the foods

image.png

nus-pe-script commented 4 days ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Food entries also not being shown when the command list-food is used

Steps to reproduce

add-food orange 200 list food

Expected outcome

Here is your food intake list: 1. orange (200 calories) at 15/11/2024 17:32:01

Actual Outcome

Food Entries for 2024-11-15: Total daily Calories: 0

Screenshot

bug 10.png

[original: nus-cs2113-AY2425S1/pe-interim#615] [original labels: type.FunctionalityBug severity.High]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for taking the time to bring this to our attention. We can replicate this and acknowledge that this missing functionality is a issue likely to impact some users. However, upon consideration, we believe that this bug only merits a Medium severity, as opposed to High.

We believe this as per the reporting guide, a High severity bug is one that "makes the product almost unusable for most users.". We believe this statement is untrue as the advertised focus and design consideration of our program was to make an effective exercise and training tracker to prepare users for their NAPFA test - not to track users food intake. Calorie/Water intake management is described in both the UG and DG as an additional functionality that may provide some self-motivated benefit to users who choose to use it, as it only provides listing functionality and doesn't tie in to other systems by design. Therefore, as a secondary design consideration / partially out-of-scope addition that doesn't directly influence the main functionality of the program, we believe that losing access to food-reporting functions only impacts the targeted userbase in the way suggested by a Medium rating - as an "inconvenience" faced by some users, who can still use the program for it's intended and advertised purpose.

We appreciate your help and time spent making our program more robust and bug-free.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** Command is a main feature of product and would affect most users of the product as a whole. All commands implemented should be able to work well.