Skip to content

Commit

Permalink
remove 'waiting for deposit' message when deposit confirmed message i…
Browse files Browse the repository at this point in the history
…s shown

fix brave#5300

Auditors: @mrose17

Test plan:
1. enable brave payments, wait up to 15 min for 'waiting for deposit' notification to appear
2. make a deposit and wait for it to be processed. the first notification should disappear when the second appears.
  • Loading branch information
diracdeltas committed Nov 2, 2016
1 parent 8d369b0 commit 5de9547
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions app/ledger.js
Original file line number Diff line number Diff line change
Expand Up @@ -1530,6 +1530,8 @@ const showNotificationPaymentDone = (transactionContributionFiat) => {
notificationPaymentDoneMessage = locale.translation('notificationPaymentDone')
.replace(/{{\s*amount\s*}}/, transactionContributionFiat.amount)
.replace(/{{\s*currency\s*}}/, transactionContributionFiat.currency)
// Hide the 'waiting for deposit' message box if it exists
appActions.hideMessageBox(addFundsMessage)
appActions.showMessageBox({
greeting: locale.translation('updateHello'),
message: notificationPaymentDoneMessage,
Expand Down

0 comments on commit 5de9547

Please sign in to comment.