nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Wrong error message for invalid current price of stock #2976

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

image.png

The error message when I give 0 as the price is image.png.

0 is a positive number so the error message is not correct.


[original: nus-cs2113-AY2122S1/pe-interim#2974] [original labels: severity.Medium type.FunctionalityBug]

theodorekwok commented 2 years ago

Team's Response

0 is not a positive number so the error found is on a wrong basis

Downgrading to VeryLow as it is purely a cosmetic thing based on the error message. The error message given is also correct and accurate. The functionality is not affected at all.

Duplicate status (if any):

--