Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Rewrite London Review of Books.js #2110

Merged
merged 3 commits into from Jan 7, 2020
Merged

Conversation

@zuphilip
Copy link
Collaborator

zuphilip commented Jan 6, 2020

This fixes #2109.

This fixes #2109.
Copy link
Collaborator

adam3smith left a comment

Two small nits. Also -- might be worthwhile to just grab the title from the page rather than the metaheader to avoid having to clean it?

London Review of Books.js Outdated Show resolved Hide resolved
London Review of Books.js Outdated Show resolved Hide resolved
zuphilip added 2 commits Jan 7, 2020
@zuphilip

This comment has been minimized.

Copy link
Collaborator Author

zuphilip commented Jan 7, 2020

Please have a look at the new version.

@adam3smith adam3smith merged commit 2918277 into zotero:master Jan 7, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adam3smith

This comment has been minimized.

Copy link
Collaborator

adam3smith commented Jan 7, 2020

Thanks!

@zuphilip zuphilip deleted the zuphilip:london-review-books branch Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.