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

<p>NACK - proto and backwrd. comp. are looking fine now, just a couple of minor adjustments needed.</p><hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/5500#discussion_r635002387">core/src/main/java/bisq/core/payment/UpholdAccount.java</a>:</p>
<pre style='color:#555'>> @@ -44,4 +44,13 @@ public void setAccountId(String accountId) {
     public String getAccountId() {
         return ((UpholdAccountPayload) paymentAccountPayload).getAccountId();
     }
+
+    public String getAccountOwner() {
+        return ((UpholdAccountPayload) paymentAccountPayload).getAccountOwner();
+    }
+
+    public void setAccountOwner(String accountOwner) {
+        if (accountOwner == null) accountOwner = "";
</pre>
<p>Please use brackets around this conditional.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/5500#discussion_r635005391">core/src/main/java/bisq/core/payment/UpholdAccount.java</a>:</p>
<pre style='color:#555'>> @@ -44,4 +44,13 @@ public void setAccountId(String accountId) {
     public String getAccountId() {
         return ((UpholdAccountPayload) paymentAccountPayload).getAccountId();
     }
+
+    public String getAccountOwner() {
</pre>
<p>Is there a reason why you don't display the account owner in <code>addFormForDisplayAccount</code>?</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/5500#discussion_r635008602">core/src/main/java/bisq/core/payment/payload/UpholdAccountPayload.java</a>:</p>
<pre style='color:#555'>> +                if (accountOwner.equals("")) {
+                    return
+                            Res.get("payment.account") + ": " + accountId + "\n" +
+                                    Res.get("payment.account.owner") + ": N/A";
+                } else {
+                    return
+                            Res.get("payment.account") + ": " + accountId + "\n" +
+                                    Res.get("payment.account.owner") + ": " + accountOwner;
+                }
</pre>
<p>Could you please decrease the intention</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/5500#pullrequestreview-662880446">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNUVLKA2NLZF3K4JP53TONXF5ANCNFSM45ACI4KA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNVZXDM4K65KZVABJXLTONXF5A5CNFSM45ACI4KKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOE6BMBPQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/5500#pullrequestreview-662880446",
"url": "https://github.com/bisq-network/bisq/pull/5500#pullrequestreview-662880446",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>