nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Don't allow for commas #752

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Don't allow for commas in the price, can be confusing for a user since you accept up to 10,000 and not stated in the user guide that it is not allowed image.png


[original: nus-cs2113-AY2223S1/pe-interim#785] [original labels: severity.Low type.FunctionalityBug]

jingwei55 commented 1 year ago

Team's Response

As stated in the user guide:

image.png

There are no commas stated in the price input. Also, the error message already stated that price is supposed to be a float, and in Java context a float only accepts the '.' symbol aside from numbers.

Duplicate status (if any):

--