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

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4308#discussion_r443655233">cli/src/main/java/bisq/cli/CliMain.java</a>:</p>
<pre style='color:#555'>> +                    out.println(reply.getFundingAddressesInfo());
+                    return;
+                }
+                case createpaymentacct: {
+                    if (nonOptionArgs.size() < 2)
+                        throw new IllegalArgumentException("no account name specified");
+
+                    var accountName = nonOptionArgs.get(1);
+
+                    if (nonOptionArgs.size() < 3)
+                        throw new IllegalArgumentException("no account number specified");
+
+                    var accountNumber = nonOptionArgs.get(2);
+
+                    if (nonOptionArgs.size() < 4)
+                        throw new IllegalArgumentException("no fiat currency specified");
</pre>
<p>These size-based checks don't actually know if it was the last argument that was ommitted. We might (or might not) go for more interesting interfaces in the future, but in this case we only know that not all arguments were passed.</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/4308#pullrequestreview-435052357">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNTPGZKTHRPHPNRD5ODRX54DZANCNFSM4N6MWJDQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNWBJ2IS5IKILURN6NDRX54DZA5CNFSM4N6MWJD2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODHXF6RI.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/4308#pullrequestreview-435052357",
"url": "https://github.com/bisq-network/bisq/pull/4308#pullrequestreview-435052357",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>