Set 2 visits on minimum visit for publisher relevancy and automatically back to 5

[bitcoinuser]

Did you search for similar issues before submitting this one? Yes

Describe the issue you encountered:

I set 2 visits on minimum visit for publisher relevancy and automatically back to 5

Expected behavior:

Keep on 2

• Platform (Win7, 8, 10? macOS? Linux distro?): Ubuntu 64
• Brave Version: Dev Channel Beta v0.12.5dev-RC1
• Steps to reproduce:
  1. Go to preferences/payment
     2.Open advanced settings
  2. Set 2 visits

Obs. See the video uploaded to understand the problem
Brave.ogv.zip

[jkup]

Also heard that dropping the number of visits clears out the ledger list. Going to look into both for this ticket.

[bitcoinuser]

Also heard that dropping the number of visits clears out the ledger list

Happened with me too.

[jkup]

Looking into this now. I suspect this started happening from this commit 42146a3

[jkup]

Update: I think the clearing of the ledger list is working as designed. If you don't have any entries with 10+ visits and you change your minimum visits to 10 they should all disappear.

Still investigating the choice not persisting!

[luixxiul]

Test plan: #5824 (comment)