[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