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

<p>NACK - Just move the one trim of the setAccountNameWithString to the PaymentMethodForm and utACk for the rest.</p><hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3573#discussion_r345691941">desktop/src/main/java/bisq/desktop/components/paymentmethods/BankForm.java</a>:</p>
<pre style='color:#555'>> @@ -240,28 +240,28 @@ public void addFormForAddAccount() {
         bankNameInputTextField = addInputTextField(gridPane, ++gridRow, Res.get("payment.bank.name"));
 
         bankNameInputTextField.textProperty().addListener((ov, oldValue, newValue) -> {
-            bankAccountPayload.setBankName(newValue);
+            bankAccountPayload.setBankName(newValue.trim());
</pre>
<p>I think the Payload objects should be POJOs without validation or verification code IMO. In that case the BankForm is an abstract class anyways and will be reused by all BankForm implementations.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3573#discussion_r345693177">desktop/src/main/java/bisq/desktop/components/paymentmethods/MoneyBeamForm.java</a>:</p>
<pre style='color:#555'>> @@ -75,7 +75,7 @@ public void addFormForAddAccount() {
 
     @Override
     protected void autoFillNameTextField() {
-        setAccountNameWithString(accountIdInputTextField.getText());
+        setAccountNameWithString(accountIdInputTextField.getText().trim());
</pre>
<p>Here it makes sense to move the trim into the PaymentMethodForm as there is no value to pollute your account name (displayed to creator only) with additional spaces. So it would help to remove unnecessary whitespaces that were added by mistake.</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/3573?email_source=notifications&email_token=AJFFTNUVHVGJORLOXIYOF6LQTPMQ3A5CNFSM4JKDPZUKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLMCNGA#pullrequestreview-316155544">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNUE2GHQWNVF2AWTXELQTPMQ3ANCNFSM4JKDPZUA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNRZC2SGG5K7IZAYLXTQTPMQ3A5CNFSM4JKDPZUKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLMCNGA.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/3573?email_source=notifications\u0026email_token=AJFFTNUVHVGJORLOXIYOF6LQTPMQ3A5CNFSM4JKDPZUKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLMCNGA#pullrequestreview-316155544",
"url": "https://github.com/bisq-network/bisq/pull/3573?email_source=notifications\u0026email_token=AJFFTNUVHVGJORLOXIYOF6LQTPMQ3A5CNFSM4JKDPZUKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLMCNGA#pullrequestreview-316155544",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>