search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Able to edit ct in finance to have positive amount #2486

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

image.png

[original: nus-cs2103-AY1920S2/pe-interim#2471]

ivanteejj commented 4 years ago

Team's Response

Thank you so much for reporting this bug! Actually we had a look at this and discussed before. We gave the user the power to edit freely the amount if that is what the user really want! Financetype cs, ct was initially implemented for the convenient of the user as it helps generated Finance name and record respective IDs. However, if the user really want to edit a FinanceType's amount, we allow it to happen as the user will know what he wants to do. But as you pointed it out, this seems to make the user more confused. An amount of ct type should never be positive. We will consider this as a nice-to-have enhancement to have restriction for negative amount only of Finance ct type.

Duplicate status (if any):

--