/translators

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

Update beck-online.js #1516

Merged
merged 1 commit into from Jan 12, 2018
+22 −20

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@zuphilip @adam3smith
@zuphilip
Collaborator

zuphilip commented Jan 8, 2018

They changed the HTML slightly, such that some elements (date, number)
were not anymore extracted. Moreover, some cases did not work anymore,
e.g. Leitsatzkartei. This commit should fix all of these.
@zuphilip
Update beck-online.js 
They changed the HTML slightly, such that some elements (date, number)
were not anymore extracted. Moreover, some cases did not work anymore,
e.g. Leitsatzkartei. This commit should fix all of these.
200deb9

@adam3smith adam3smith merged commit c7ce0da into zotero:master Jan 12, 2018
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@adam3smith

This comment has been minimized.

Show comment
Hide comment
@adam3smith

adam3smith Jan 12, 2018

Collaborator

Great, thanks!
Collaborator

adam3smith commented Jan 12, 2018

Great, thanks!

@zuphilip zuphilip deleted the zuphilip:update-beck branch Jan 12, 2018

GuyAglionby added a commit to GuyAglionby/translators that referenced this pull request Jan 28, 2018

@zuphilip @GuyAglionby
Update beck-online.js (#1516) 
They changed the HTML slightly, such that some elements (date, number)
were not anymore extracted. Moreover, some cases did not work anymore,
e.g. Leitsatzkartei. This commit should fix all of these.
6713e8d

psisquared2 added a commit to psisquared2/translators that referenced this pull request Feb 8, 2018

@zuphilip @psisquared2
Update beck-online.js (#1516) 
They changed the HTML slightly, such that some elements (date, number)
were not anymore extracted. Moreover, some cases did not work anymore,
e.g. Leitsatzkartei. This commit should fix all of these.
0a40124
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment