<p><b>@ManfredKarrer</b> approved this pull request.</p>

<p>ACK</p>
<p>Thanks a lot for addint it! Dont forget to file a compensation request at <a href="https://github.com/bisq-network/compensation/issues">https://github.com/bisq-network/compensation/issues</a></p><hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/1831#discussion_r228754948">core/src/main/java/bisq/core/payment/payload/PaymentMethod.java</a>:</p>
<pre style='color:#555'>> @@ -227,6 +229,9 @@ public PaymentMethod(String id, long maxTradePeriod, @NotNull Coin maxTradeLimit
                     ALI_PAY = new PaymentMethod(ALI_PAY_ID, DAY, maxTradeLimitLowRisk),
                     WECHAT_PAY = new PaymentMethod(WECHAT_PAY_ID, DAY, maxTradeLimitLowRisk),
</pre>
<p>Is that really a low risk payment method? Those are the ones where chargeback risk is super low or even stated that it is not possible to do chargeback. If so great, otherwise maybe better to use the mid risk parameter.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/1831#discussion_r228755234">core/src/main/java/bisq/core/payment/payload/PaymentMethod.java</a>:</p>
<pre style='color:#555'>> @@ -227,6 +229,9 @@ public PaymentMethod(String id, long maxTradePeriod, @NotNull Coin maxTradeLimit
                     ALI_PAY = new PaymentMethod(ALI_PAY_ID, DAY, maxTradeLimitLowRisk),
                     WECHAT_PAY = new PaymentMethod(WECHAT_PAY_ID, DAY, maxTradeLimitLowRisk),
</pre>
<p>Ah I just read the comment in the issue. Seems its really low risk. So lets keep it.</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/1831#pullrequestreview-169099029">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtnj1N1UPWtWHNNAeT192x_Q4ERu-ks5upex7gaJpZM4X99qk">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtm5ARLYfEp0XLFX6-3b8uAsQS3_Wks5upex7gaJpZM4X99qk.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://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ManfredKarrer approved #1831"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq/pull/1831#pullrequestreview-169099029"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/1831#pullrequestreview-169099029",
"url": "https://github.com/bisq-network/bisq/pull/1831#pullrequestreview-169099029",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "@ManfredKarrer approved 1831",
"sections": [
{
"text": "ACK\r\n\r\nThanks a lot for addint it! Dont forget to file a compensation request at https://github.com/bisq-network/compensation/issues",
"activityTitle": "**Manfred Karrer**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@ManfredKarrer",
"facts": [

]
}
],
"potentialAction": [
{
"targets": [
{
"os": "default",
"uri": "https://github.com/bisq-network/bisq/pull/1831#pullrequestreview-169099029"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 402119332\n}"
}
],
"themeColor": "26292E"
}
]</script>