nus-pe-bot
commented
6 months ago Team's Response
No details provided by team.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Inaccurate error message for low_stock
Not mentioned in UG what is the type for AMOUNT. User cant know why he cannot use string representation of a number. Perhaps can give an error message which says strings are not supposed to be used
[original: nus-cs2113-AY2324S2/pe-interim#393] [original labels: severity.Low type.FeatureFlaw]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
We admit that this message could have been changed to a clearer one. Since it's not mentioned in UG that amount has to be integer.
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)
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
Is amount supposed to be an integer? It is not mentioned in the UG that it is supposed to be an integer and gives an error when strings, doubles, or any other combinations of different types are used. Please be more clear.