Forum Replies Created
-
AuthorPosts
-
alexg
KeymasterYep, it’s explained in the coin adapter settings.
For reference, fixed fees are an amount that is charged on each transaction. Proportional means that the transacted amount is multiplied by the fee “proportion” and therefore the fee paid for the transaction is a fraction of the total transacted amount.
From the documentation’s glossary section:
Fees are subtracted from user accounts and remain with the wallet. For every coin, there is one wallet that is split between all users with the has_wallets capability. In the case of withdrawals, the fees are first used to pay for the blockchain’s network fees, and any remaining fees stay with your site’s wallet. In the case of internal transfers between users, the entirety of the paid fees is subtracted from the sender’s balance and remains with your wallet, since there is no blockchain transaction. When you visit the *coin adapters* screen in your admin area, you can see the total of user balances for each coin, and the balance of the wallet. These amounts do not have to be the same. As users perform transactions and pay fees, these fees will be subtracted from the user balances, but not from the wallet balance.
alexg
KeymasterI did some more tests and still cannot figure out why your first address did not work. Withdrawals in the CoinPayments adapter are a stable feature that was developed a long time ago and has been working fine over the years; it’s unlikely that there’s any bug in the adapter. Did you attempt a test withdrawal from the platform to your address?
alexg
KeymasterThank you.
I understand your argument, but I have not reached a decision on what to do on this. Doing so would take too much time right now that I do not have.
In the meantime, you could use text to explain to your users that they cannot deposit/withdraw custom coins, or simply do not use custom coins.
When I arrive at a decision I will let you know.
with regards
alexg
KeymasterHere’s an idea:
Login to CoinPayments, and attempt a test withdrawal to the first address, using the CoinPayments website: https://www.coinpayments.net/index.php?cmd=acct_balances&action=withdraw&coin=BTC
Does the platform accept the address? Or do you get the same error?
alexg
KeymasterThe first address you gave is a P2PKH and the second one is P2SH. To read up more on address types, see https://en.bitcoin.it/wiki/Address
If you visit the addresses in a block explorer it usually tells you the type.
https://www.blockchain.com/btc/address/16YCtPK5gWewAbcauTmfsWmUMLJnkn7u2v
https://www.blockchain.com/btc/address/3BMEX3KRRHCGrXQZGpjjWyNeZBNGwe9wV1The problem is that both should work. At least now we know that sending to P2SH addresses works for you.
Did you check for non-printable characters in your transactions? How are you inputting the address? QR code or paste from clipboard? I can’t think of anything else right now. This is a weird error.
alexg
KeymasterOK then, it doesn’t look like a bech32 address. Without having the exact address I cannot verify if it is correct. Did you try withdrawing to several addresses or just one?
In any case, this error comes from the platform, not the plugin.
Check your withdrawal history in CoinPayments, and see if there is any more information there.
Also, check if there were there any whitespaces in the address, or even unprintable characters? Several unicode characters, such as non-breaking white spaces (NBSPs), can sometimes be copied together with the address and can cause havoc.
You may be able to get more information by contacting CoinPayments support. Mention the CPID and they should be able to tell you why the withdrawal failed. If the withdrawal has failed, then the CPID should be in place of the TXID. Additionally, if you login to the platform and go to the withdrawal history, you should be able to see the CPID for your transaction.
Hope this helps.
with regards
alexg
KeymasterHello,
This error does not originate from the plugin but from the wallet backend.
The CoinPayments API returns “That is not a valid address for that coin!”. Check the format of your withdrawal address (P2PKH, P2SH, or Bech32). Is it compatible with CoinPayments? For example, I am not certain if Bech32 is supported for Bitcoin transactions on CoinPayments.
Try a different address format and see if it makes a difference. Let me know what you found.
with regards
alexg
KeymasterThank you. Good job.
I also want to do some work on styling the UIS for mobile/responsive, but it’s of secondary priority right now compared to some other stuff. Ideally if you can write your own rules then you will match your theme better than I can. But mobile styling will come to the plugin eventually 🙂
with regards
alexg
KeymasterHello,
@bigblue Thanks for letting me know. There are no settings. Did you check with one theme or many? This sounds a lot like a CSS issue. Can you show me a screenshot or give me access to a site?
@jamiej I am not sure why you reinstalled wordpress, at the very least you could downgrade to the previous version of the Exchange extension. These things are usually CSS issues. It is near impossible to predict what CSS rules a theme has, I’m just aiming to make something that works with a few standard themes, glad you found a solution. The graph already has the red/green colors that you set in customizer. In a later version I may add more styling controls in customizer.with regards
alexg
KeymasterHello,
What I am talking about is not a bigger change, only a more general and simpler one. At this time I do not see a need to work on this extension. Once I decide to work on this I will let you know.
with regards
alexg
KeymasterNow that version
1.3.0
of the Exchange extension is out, and the chart is implemented using your suggestion, the TradingView lightweight-charts library, the chart is now responsive if you do not specify width/height. When the window resizes, the charts are redrawn to the new available container size.alexg
KeymasterHello again,
OK it is done! In version
1.3.0
of the Exchange extension I’ve used lightweight-charts from TradingView, replacing the old implementation that used techan.js.Thank you for the suggestion, it feels much better now!
This is not to say that techan.js is not a good library, but I never figured out how to control the visible frame using the mouse with it.
with regards
alexg
KeymasterThe above styling is now part of version
1.3.0
of the Exchange. The dropdowns should appear more uniform across various themes now.alexg
KeymasterOK this requires more thought than I initially believed:
Essentially you want to control which coins are allowed to do any one of the operations move, deposit, withdraw. This requires big changes. If it is to be done, it has to be done for all coins, and in a way that will not require more changes two weeks down the road.
Can you tell me more about your use case? What are you using custom coins for? Is it some type of store credit that does not make sense to deposit/withdraw?
Thank you.
alexg
KeymasterHello,
Thank you for this. I had no idea that the TradingView chart is usable for custom data.
This won’t be easy, mainly because I will first need to untangle the techan chart from the code, which is a mess.
But it will be necessary. Having a TV chart will be a big improvement, so I will try to implement it soon. I will update you on this thread.
thanks again.
with regards
-
AuthorPosts