search

t-l-xin / pe

0 stars 0 forks source link

Program does not check for unrealistically small stock alert thresholds #4

Open t-l-xin opened 2 years ago

t-l-xin commented 2 years ago

does not handle very small stock levels.JPG

This amount is unrealistically small for any stock and can essentially be 0 already.

If a stock were to fall to a small e.g. 0.1kg and requires restocking, this setting may not allow the stock to be alerted at all.

shown below

stock issue.JPG

nus-pe-bot commented 2 years ago

Team's Response

The amount is based in kg, so it is extremely unrealistic for a user to key in any value with precision more than 3dp

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: A careless user might accidentally type wrong with the slip of the finger and include many decimal points. This will then result in the situation as described in my original bug post, hence there is still a need to implement checking for long inputs.