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

Need new strings to handle overdue payments #7524

Merged
merged 1 commit into from
Mar 10, 2017
Merged

Need new strings to handle overdue payments #7524

merged 1 commit into from
Mar 10, 2017

Conversation

mrose17
Copy link
Member

@mrose17 mrose17 commented Mar 7, 2017

Fixes #7078

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Test Plan:

  • Create a new wallet
  • Stop browser
  • Set reconcileStamp ledger-state.json to the current timestamp or more than a day in the past.
  • Verify that the text indicates "is due" an "is coverdue" instead of suggesting a future event.

@mrose17 mrose17 added this to the 0.13.6 milestone Mar 7, 2017
@mrose17 mrose17 self-assigned this Mar 7, 2017
@mrose17 mrose17 requested a review from bsclifton March 7, 2017 23:57
@@ -198,7 +200,7 @@ ledgerRecoveryFailedTitle=Recovery Failed
ledgerRecoveryFailedMessage=Please re-enter keys or try different keys.
ledgerRecoveryNetworkFailedTitle=Network Error
ledgerRecoveryNetworkFailedMessage=Please check your internet connection and try again.
ledgerRecoveryContent=The balance of the recovered wallet will be transferred to your new Brave wallet. The old wallet will still exist as an empty wallet.
ledgerRecoveryContent=Your previous wallet will now be used. Your new wallet will be discarded.
Copy link
Member

Choose a reason for hiding this comment

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

Just to confirm:
@mrose17 when you do a restore, it'll use the previous wallet (what you just restored), right? So the old text here (about transferring the balance to your new wallet) was wrong?

Copy link
Member Author

Choose a reason for hiding this comment

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

correct. wallet recovery is now actually "recovery of old wallet" rather than "transfer the balance of the old wallet to the new wallet"

@NejcZdovc
Copy link
Contributor

Regarding payment history button I think this is not relevant any more, because we removed payment history button and added only the icon, more in #7532

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

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

++

@bsclifton
Copy link
Member

bsclifton commented Mar 10, 2017

@NejcZdovc I believe the strings are also shown when you click the icon (and it shows the statements)
screen shot 2017-03-10 at 1 31 05 pm

Although, if that new icon isn't clickable then the strings may not be needed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants