mastercoin-MSC / masterchest-wallet

Masterchest Wallet
MIT License
15 stars 13 forks source link

No error when trying to sell more than an address has #12

Closed ripper234 closed 10 years ago

ripper234 commented 10 years ago

I tried to sell 10 TMSC from an address that only had 1 TMSC. When I clicked on the 'Sell' button after fliling out the details, nothing happened.

Instead, the sell button should be unclickable, and an error should be displayed in the dialog that explains there aren't enough TMSC in that address.

zathras-crypto commented 10 years ago

Not so sure about sell being unclickable - I put a change into alpha 0.3 that addresses this slightly differently. In alpha 0.3 if you create a sell higher than your available balance and click sell, your sell will now be automatically adjusted down to your available balance and you will be advised that the sell was over your balance and your sell has been automatically adjusted down & to click sell again if you accept the amended details. Suffiicient?

ripper234 commented 10 years ago

Sounds better than my solution. Closing.