-
Notifications
You must be signed in to change notification settings - Fork 862
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
Hides sites with 0% attention from the A-C table #1032
Conversation
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.
If I modify the database to specify 0%, it still shows in the main list.
(Image sent via Slack. Github not processing images right now for some reason)
@jasonrsadler - I updated the bat-native ledger PR to add a |
@jasonrsadler @NejcZdovc rebased with master - native-ledger updates are now included in this PR |
a8cd36d
to
a8012c3
Compare
Hides sites with 0% attention from the A-C table
Hides sites with 0% attention from the A-C table
Fixes: brave/brave-browser#2365
Native-ledger: brave-intl/bat-native-ledger#199
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:
publisher_info_db
can be edited directly to add someReviewer Checklist: