<p><b>@ripcurlx</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/2413#discussion_r256858118">core/src/main/java/bisq/core/payment/payload/PaymentMethod.java</a>:</p>
<pre style='color:#555'>>              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)
</pre>
<p>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 <code>riskFactor</code> of 8 as the <code>DEFAULT_TRADE_LIMIT_VERY_LOW_RISK.value</code> is 1 BTC.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq/pull/2413#pullrequestreview-203774700">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZttYZzO-zRo1CWKYg3eXbz4mc37Jlks5vNXRwgaJpZM4a3ntC">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtg87_9IFBrpVeFTEHYHRcdgRxoWvks5vNXRwgaJpZM4a3ntC.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq","title":"bisq-network/bisq","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ripcurlx commented on #2413"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq/pull/2413#pullrequestreview-203774700"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/2413#pullrequestreview-203774700",
"url": "https://github.com/bisq-network/bisq/pull/2413#pullrequestreview-203774700",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>