search

mastercoin-MSC / masterchest-wallet

Masterchest Wallet
MIT License
15 stars 13 forks source link

Validate addresses on the fly before clicking send #35

Open ripper234 opened 10 years ago

ripper234 commented 10 years ago

When sending / using the DEx, we should validate an address whenever the Recipient Address field changes value (OnChange or something).

In real time, when the text contains a value that is not a valid bitcoin address, we should show a red error indicator below the address, and the Send button should be disabled.

zathras-crypto commented 10 years ago

I'm against this for now. My reasoning is load. At some point I'll move address validation into dotNET code which would make this feasible but right now it's done by the 'validateaddress' RPC call to bitcoind/qt. Making an RPC call every time a character changes in that field (eg as you say onchange) will slow down the UI considerably.

ripper234 commented 10 years ago

Of course this should be done only when it can be done without RPC calls!

Heh, it appears that this old question is mine.

Perhaps the author of this library can help.