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

Add unit tests for ledger backup and minimum visits #5887

Closed
jkup opened this issue Nov 28, 2016 · 1 comment · Fixed by #6529
Closed

Add unit tests for ledger backup and minimum visits #5887

jkup opened this issue Nov 28, 2016 · 1 comment · Fixed by #6529
Labels
automated-tests feature/rewards

Comments

@jkup
Copy link
Contributor

jkup commented Nov 28, 2016

Following this comment from @bsclifton #5824 (comment)

We should add some testing around two recent fixes:

  1. Advanced ledger settings stopped persisting Unable to change minimum visits in ledger advance settings #5637
  2. Clean up formatting for ledger wallet backup Clean up formatting for bitcoin wallet backup #5862
@jkup jkup added this to the 0.13.1 milestone Nov 28, 2016
@jkup jkup self-assigned this Nov 28, 2016
@jkup jkup mentioned this issue Nov 28, 2016
Merged
4 tasks
@willy-b willy-b mentioned this issue Nov 30, 2016
Closed
@jkup
Copy link
Contributor Author

jkup commented Dec 15, 2016

bumping until the new react unit test dependencies make it into master

@jkup jkup modified the milestones: 0.13.2, 0.13.1 Dec 15, 2016
@jkup jkup modified the milestones: 0.13.1, 0.13.2 Dec 30, 2016
This was referenced Jan 5, 2017
Closed
Merged
@mrose17 mrose17 removed this from the 0.13.1 milestone Jan 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
automated-tests feature/rewards
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Split PaymentsTab into its own component brave/browser-laptop
3 participants
@mrose17 @jkup @luixxiul