Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upelife translator issues #1719
Comments
adam3smith
self-assigned this
Sep 27, 2018
adam3smith
added
Major
Error
Difficulty: Easy
labels
Sep 27, 2018
adam3smith
closed this
in
#1754
Sep 27, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
tmchartrand commentedAug 20, 2018
The site layout for eLife seems to have changed at some point, and the regex no longer matches.
For an example url, see https://elifesciences.org/articles/16553
Simply adding "|articles" to the regex fixed the issue for me.
Full target regex:
^https?://(elife\.)?elifesciences\.org/(content/|search|category/|browse|archive|articles)
Additionally, there's some code that apparently used to clean up extra authors, but now removes all:
I'm unable to find an example of the old scheme where this was necessary, but commenting it out fixes the import for me.
Finally, the pdf attachments are no longer found. This currently relies on the embedded metadata translator, and it seems some custom code is needed for the new layout.