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

<p>utACK</p><hr>

<p>In <a href="https://github.com/bisq-network/bisq-core/pull/138#discussion_r204237080">src/main/java/bisq/core/payment/payload/F2FAccountPayload.java</a>:</p>
<pre style='color:#555'>> +
+
+    ///////////////////////////////////////////////////////////////////////////////////////////
+    // API
+    ///////////////////////////////////////////////////////////////////////////////////////////
+
+    @Override
+    public String getPaymentDetails() {
+        return "Face to Face - Contact: " + contact + ", city: " + city + ", additional information: " + extraInfo;
+    }
+
+
+    @Override
+    public String getPaymentDetailsForTradePopup() {
+        // We don't show here more as the makers extra data are the relevant for the trade. City has to be anyway the
+        // same for maker and taker.
</pre>
<p>While it's true that the city has to be the same it might just be nice to have that confirmation that you're in the right city in case multiple f2f have been set up.</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-core/pull/138#pullrequestreview-139284255">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtu00YpD7WppEebITypjvpW_n9AfHks5uJGOwgaJpZM4VZYnn">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtolYc4an68uGFKWZalGo4LgAHf9Nks5uJGOwgaJpZM4VZYnn.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-core","title":"bisq-network/bisq-core","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-core"}},"updates":{"snippets":[{"icon":"PERSON","message":"@sqrrm approved #138"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-core/pull/138#pullrequestreview-139284255"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq-core/pull/138#pullrequestreview-139284255",
"url": "https://github.com/bisq-network/bisq-core/pull/138#pullrequestreview-139284255",
"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": "@sqrrm approved 138",
"sections": [
{
"text": "utACK",
"activityTitle": "**sqrrm**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@sqrrm",
"facts": [

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