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

Adds a RIS citation tool for the Publication show page. #773

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bwatson78
Copy link
Contributor

  • app/controllers/concerns/hyrax/works_controller_behavior.rb: removes this full file override after moving the two methods overwritten to app/controllers/hyrax/publications_controller.rb.
  • app/controllers/hyrax/publications_controller.rb: takes in overrides of #additional_response_formats and #permissions_changed? and creates #respond_to_ris that largely mimics Hyrax' #respond_to_endnote logic.
  • app/models/concerns/self_deposit/ris_behavior.rb: initiates a module holding all of the logic needed to pull and process the necessary RIS text.
  • app/models/solr_document.rb: includes the module referenced above.
  • app/views/hyrax/base/_citations.html.erb: adds the link for RIS that is using the same format response processing as Endnote.
  • config/initializers/mime_types.rb: adds .ris as an expected extension.
  • spec/*: checks for link presence and correct RIS file processing.

image

@bwatson78 bwatson78 added enhancement New feature or request Backend labels Feb 7, 2025
@bwatson78 bwatson78 requested a review from alexBLR February 7, 2025 14:57
@bwatson78 bwatson78 self-assigned this Feb 7, 2025
Copy link

codeclimate bot commented Feb 7, 2025

Code Climate has analyzed commit f6c915f and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 88.7% (50% is the threshold).

This pull request will bring the total coverage in the repository to 78.8% (5.2% change).

View more on Code Climate.

@bwatson78 bwatson78 mentioned this pull request Feb 7, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant