-
Notifications
You must be signed in to change notification settings - Fork 900
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds error flow for corrupted wallet #927
Conversation
cfe9fa8
to
f6cfc80
Compare
f6cfc80
to
a0980b7
Compare
@jasonrsadler this was intentionally added, so that user can do backup and recovery, we could hide add funds button, let me do that |
It might be easier to put in a link to the backup/recover overlay. wdyt? |
a0980b7
to
dabc4b9
Compare
Looks good! I hate to ask one more thing, but can we make the error go away after recovering? |
dabc4b9
to
0aefbc1
Compare
0aefbc1
to
db1ca86
Compare
@jasonrsadler done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
Adds error flow for corrupted wallet
Resolves brave/brave-browser#2183
Native implementation: brave-intl/bat-native-ledger#182
Submitter Checklist:
npm test brave_unit_tests && npm test brave_browser_tests
) ongit rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist: