search

RiaVora / pe

0 stars 0 forks source link

Don't allow for commas #3

Open RiaVora opened 1 year ago

RiaVora 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

nus-pe-script 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.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]