[bisq-network/bisq] Increase trade limit. Make trade limit a DAO parameter (#2413)
Christoph Atteneder
notifications at github.com
Thu Feb 14 14:36:32 UTC 2019
ripcurlx commented on this pull request.
> return Coin.parseCoin("4");
+
+ // We use the class field maxTradeLimit only for mapping the risk factor.
+ long riskFactor;
+ if (maxTradeLimit == DEFAULT_TRADE_LIMIT_VERY_LOW_RISK.value)
There is a problem that if a low risk payment method (e.g. altcoin) has maxTradeLimit already set to two BTC it will be set as `riskFactor` of 8 as the `DEFAULT_TRADE_LIMIT_VERY_LOW_RISK.value` is 1 BTC.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/2413#pullrequestreview-203774700
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190214/8a325c70/attachment.html>
More information about the bisq-github
mailing list