dinhnhobao / pe

0 stars 0 forks source link

Not my first time opening Cargo but gets welcoming message #7

Open dinhnhobao opened 4 years ago

dinhnhobao commented 4 years ago

No details provided. image.png

nus-pe-bot commented 4 years ago

Team's Response

Hi there.

As Calgo is a Calorie-Tracking App, users should set their Caloric Goal on the first time in the application, and thus never see this message again unless they delete the data.

Bug Type: Feature Flaw as it does not change the message to no longer indicate that "This is your first time", and is therefore purely cosmetic, and does not create subsequent bugs. Severity (if any): very low because this is purely cosmetic and no usage effects (if any).

Items for the Tester to Verify

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: Your issue matches most of the descriptions of FunctionalityBug: image.png


:question: Issue severity

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

Reason for disagreement: image.png Your code clearly fails to handle the case when the user did not set his or her goals, so it should be at least a severity.Low depending on whether the goal is critical for the product. The error message results from your wrong handling of logic, so it's absolutely not a cosmetic issue with the UI message at all. severity.VeryLow is only for cosmetics, image.png, however, your code is wrong in the logic.