Skip to content

Commit

Permalink
Merge pull request #1822 from ManfredKarrer/add-cash-depoist-warning
Browse files Browse the repository at this point in the history
Add warning popup for cash deposit
  • Loading branch information
ripcurlx authored Oct 29, 2018
2 parents 1350413 + cc446fb commit 55133ef
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
3 changes: 3 additions & 0 deletions core/src/main/resources/i18n/displayStrings.properties
Original file line number Diff line number Diff line change
Expand Up @@ -2213,6 +2213,9 @@ payment.limits.info=Please be aware that all bank transfers carry a certain amou
\n\
Please note that there are no limits on the total number of times you can trade.

payment.cashDeposit.info=Please confirm your bank allows you to send cash deposits into other peoples' accounts. \
For example, Bank of America and Wells Fargo no longer allow such deposits.

payment.f2f.contact=Contact info
payment.f2f.contact.prompt=How you want to get contacted by the trading peer? (email address, phone number,...)
payment.f2f.city=City for 'Face to face' meeting
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@
import bisq.core.app.BisqEnvironment;
import bisq.core.locale.Res;
import bisq.core.payment.AccountAgeWitnessService;
import bisq.core.payment.CashDepositAccount;
import bisq.core.payment.ClearXchangeAccount;
import bisq.core.payment.F2FAccount;
import bisq.core.payment.HalCashAccount;
Expand Down Expand Up @@ -298,6 +299,13 @@ private void onSaveNewAccount(PaymentAccount paymentAccount) {
.actionButtonText(Res.get("shared.iUnderstand"))
.onAction(() -> doSaveNewAccount(paymentAccount))
.show();
} else if (paymentAccount instanceof CashDepositAccount) {
new Popup<>().information(Res.get("payment.cashDeposit.info"))
.width(700)
.closeButtonText(Res.get("shared.cancel"))
.actionButtonText(Res.get("shared.iConfirm"))
.onAction(() -> doSaveNewAccount(paymentAccount))
.show();
} else {
doSaveNewAccount(paymentAccount);
}
Expand Down

0 comments on commit 55133ef

Please sign in to comment.