Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

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 #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

1 comment on commit 5de9547

@mrose17
Copy link
Member

@mrose17 mrose17 commented on 5de9547 Nov 2, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm.

Please sign in to comment.