nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months ago

would be good to include $ sign for profits since sale price and cost price both has the sign

_{[original: nus-cs2113-AY2324S2/pe-interim#1792] [original labels: type.FeatureFlaw severity.Medium]}

YHWong20 commented 6 months ago

Team's Response

Thanks for the bug report. I agree that it would be good to include the $ sign for profits.

However, I disagree with the Medium severity. It is obvious that this is a cosmetic issue, which should deserve VeryLow, as seen below.

I hope you understand my decision to downgrade this bug's severity from Medium to VeryLow.