gwu-libraries / launchpad

A django based system that provides a stable URL for every item in the library's catalog. Various discovery services will link to these URLs. The page for each item will in turn link out to various other resources that provide methods for accessing the content of the items.
MIT License
15 stars 9 forks source link

consider showing openurl.query_string information, if present #237

Closed cummingsm closed 11 years ago

cummingsm commented 11 years ago

When json data has openurl.query_string, should we display the request elements openurl.params?

example: http://findit.library.gwu.edu/item/5162329.json?ctx_ver=Z39.88-2004&ctx_enc=info%3Aofi%2Fenc%3AUTF-8&rfr_id=info:sid/summon.serialssolutions.com&rft_val_fmt=info:ofi/fmt:kev:mtx:journal&rft.genre=article&rft.atitle=Renal+metabolism+of+acrylonitrile+to+cyanide%3A+In+vitro+studies&rft.jtitle=PHARMACOLOGICAL+RESEARCH&rft.au=Mostafa%2C+AM&rft.au=Abdel-Naim%2C+AB&rft.au=Abo-Salem%2C+O&rft.au=Abdel-Aziz%2C+AH&rft.date=1999-08-01&rft.pub=ACADEMIC+PRESS+LTD&rft.issn=1043-6618&rft.volume=40&rft.issue=2&rft.spage=195&rft.epage=200&rft.externalDBID=n%2Fa&rft.externalDocID=000081871800017

cummingsm commented 11 years ago

Dan will consider this further, and decided to move it from m10 to m11

cummingsm commented 11 years ago

Josh/Laura discussed a bit more. The info needed may be in the referrer value of the http session header

lwrubel commented 11 years ago

All links to launchpad now pass the OpenURL parameters.

Print holdings only: http://bit.ly/10mcELC Print holdings in hidden section under online holding: http://bit.ly/XJuKTr No print or online, link to launchpad in bottom half of screen: http://bit.ly/W4K7qG

lwrubel commented 11 years ago

Looks like Launchpad is putting this citation into the page already? Not sure if that is supposed to be the case-- it looks like it needs some formatting and context, although the citation parsing is pretty good.

I'm going to turn this off in the 360 Link "Browse this journal" and "Locate Journal" links for now. I will leave it on for the third case in the previous list of examples, the link in the lower half of the screen.

Here's a screenshot showing what's happening. LaunchpadCitation

dchud commented 11 years ago

Lost the context, moving on-