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

use updated ledger-publisher #3613

Closed
mrose17 opened this issue Aug 31, 2016 · 2 comments
Closed

use updated ledger-publisher #3613

mrose17 opened this issue Aug 31, 2016 · 2 comments

Comments

@mrose17
Copy link
Member

mrose17 commented Aug 31, 2016

to get the latest curated list from ledger-publisher@0.8.57

cf., brave/ledger-publisher#7

@mrose17 mrose17 added this to the 0.11.6dev milestone Aug 31, 2016
@mrose17 mrose17 self-assigned this Aug 31, 2016
@mrose17 mrose17 modified the milestones: 0.11.7dev, 0.11.6dev Aug 31, 2016
@alexwykoff
Copy link
Contributor

Anything for QA to check with this update?

@mrose17
Copy link
Member Author

mrose17 commented Sep 4, 2016

the issue came up last week when i noticed that brendan had a ledger-synopsis.json file that was rather odd. the file was syntatically correct, but in a format not used (ever) by the code, as required fields were missing.

to test, you can quit the browser, manually edit ledger-synopsis*.json to exclude the window from one or more publishers, and then start the browser.

you should see the publishers still be present in the preferences panel, and when you visit those publishers, they should start updating.

i have no idea how to automate this, by the way... sorry!

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

No branches or pull requests

2 participants