Skip to content
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 Google Patents.js #1755

Merged
merged 1 commit into from Oct 4, 2018

Conversation

Projects
None yet
2 participants
@hpxing
Copy link
Contributor

hpxing commented Sep 28, 2018

fix pdf url

Update Google Patents.js
fix pdf url
@adam3smith

This comment has been minimized.

Copy link
Collaborator

adam3smith commented Sep 29, 2018

do you have a link where that's needed? I don't have an example for the relative link, but typically they'd include a beginning slash?

@adam3smith adam3smith added the waiting label Sep 29, 2018

@hpxing

This comment has been minimized.

Copy link
Contributor Author

hpxing commented Oct 3, 2018

just the test case https://patents.google.com/?q=book&oq=book
without the / the pdf will not attached correctly

@adam3smith adam3smith merged commit d769ac3 into zotero:master Oct 4, 2018

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 Oct 4, 2018

perfect, thanks, that's what I needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.