Zhilin-Huang / pe

0 stars 0 forks source link

Wrong response when addf command parameters are wrong #9

Open Zhilin-Huang opened 4 years ago

Zhilin-Huang commented 4 years ago

Screen Shot 2020-04-15 at 12.53.51.png

I believe the addf command does not have anything to do with weight. Not sure why it says 'Invalid index of weight! Please check and try again.' when the command format is not followed.

This causes occasional inconvenience to users when they enter the command wrongly (which could be rather common as it is easy to mistake -- for - in the command) but they can continue to use the product.

nus-pe-bot commented 4 years ago

Team's Response

The usability is not affected in this case. This error might be caused when there is already an error in the user’s weight such as empty weight after user accidentally deleted all weight from the record. Thus the system might prompt the user to check the weight prior to using anymore functions in the program.

We can rectify this small UI issue by making the statement or intuitive for the user based on the command input if required.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This is undeniably a wrong response as it does not even make sense to respond with anything related to weight when the command has nothing to do with weight.

This also does not only happen when there is no weight recorded in the application:

Screen Shot 2020-04-20 at 16.38.12.png

This bug should not be rejected.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]