nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Transfer command is not working #1026

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Switched to the other wallet with no balance, managed to transfer and received negative balance

image.png


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

AmethystQ commented 1 year ago

Team's Response

We've discussed about whether negative values should be allowed in the program. On one hand, negative values represent you are in dept. On the other hand, it is sometimes confusing since some users don't expect that they have debts. I think it's better to give a prompt to ask users whether they accept negative values.

Duplicate status (if any):

--