[dex] DEX-locked funds awareness
See original GitHub issueThat might be something that was already mentioned/discussed somewhere, but I didn’t find and decided it is worth mentioning,
When I’m placing an order on DEX-window through Decrediton (on testnet), it isn’t reflected in Decrediton itself (an order is already booked, locked balance in Decrediton = 0):
that results in a bit of confusion when using same wallet for other things like sending transactions, because it shows all the funds on my “Balance” when I’m trying to send one:
but when I try to fill in all the info I get the error below, and I can only guess that it happens because I have 60 DCR locked in an order on DEX out of 119.something shown on screenshot above in my mixed acc1 (canceling DEX order removes this issue, that’s why I’m pretty confident it is the root cause for Insufficient funds
error I’m seeing; specifying lower amount to send also removes the error):
Issue Analytics
- State:
- Created 10 months ago
- Comments:7 (4 by maintainers)
Top GitHub Comments
Broadly speaking, Decrediton has no clue what DEX is doing, except indirectly by observing the affects on dcrwallet, which include balance and locked UTXO changes. For Decrediton to figure out if there are active orders, it presently requires logging into DEX (and updating the core adapter with a method to get the exchange map with orders, the easy part). We can probably make some changes to
client/core.Core
to make this info available to consumers like Decrediton prior to logging in. We can look at that after https://github.com/decred/dcrdex/pull/1903Yeah, messaging is what I think could be improved there.
Maybe messaging for this could be improved in Decrediton itself: when Decrediton wallet is started and there are open orders on Decrediton-embedded-DEX it could prompt the user to login directly transferring him onto DEX login page, any reason for not to do so ?