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

Add citation for SCSB Records #4012

Closed
3 of 5 tasks
christinach opened this issue May 6, 2024 · 3 comments
Closed
3 of 5 tasks

Add citation for SCSB Records #4012

christinach opened this issue May 6, 2024 · 3 comments
Assignees
Labels
feature Updates on existing feature or a new feature request PO (high value) PO reviewed and high value

Comments

@christinach
Copy link
Member

christinach commented May 6, 2024

User story

As a researcher who will be citing SCSB records I want to easily copy and paste a citation into my document

Acceptance criteria

  • Display 'Cite' button
  • Update the code to build the citations for SCSB records
  • I can successfully use this feature with Voice Over in Safari.
  • I have checked this feature using one of the following browser extensions and confirmed that it does not add any level A or level AA issues:
    • axe dev tools
    • Lighthouse
    • Wave
  • ...

Concrete example

See the title at this link from Columbia's holdings (SCSB) as an example. You cannot cite, nor export to a bibliographic management system (e.g. Zotero).

Implementation notes, if any

Related to #4124

@christinach christinach added the feature Updates on existing feature or a new feature request label May 6, 2024
@kevinreiss kevinreiss changed the title Add citation for Senior Theses Add citation for Non-Alma Derived Records May 17, 2024
@caroldh caroldh added the PO (high value) PO reviewed and high value label Aug 12, 2024
@christinach christinach changed the title Add citation for Non-Alma Derived Records Add citation for SCSB Records Aug 19, 2024
@maxkadel maxkadel self-assigned this Oct 3, 2024
@maxkadel
Copy link
Contributor

maxkadel commented Oct 7, 2024

I tried citeproc-ruby and ran into some problems.
a) I could not figure out why, but when using the MLA format, it did not add the delimiter between the last and first names. This could be a bug or driver error?
b) The publisher is not in the Solr record without the place of publication, which is needed for most MLA book citations.
c) Again, this could be my own error, but I can't get all the fields I need without another call to Solr. At that point in the code, the fields included in the document are very minimal, and the semantic values, which should include the citations fields, do not seem to at this point. I don't know if this is a loading order issue or what?

@maxkadel
Copy link
Contributor

maxkadel commented Oct 7, 2024

The .ris extension does work for SCSB items.

@maxkadel
Copy link
Contributor

Closed by #4427

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Updates on existing feature or a new feature request PO (high value) PO reviewed and high value
Projects
None yet
Development

No branches or pull requests

4 participants