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

Move riiif dependent code to the generated riiif initializer #6495

Merged
merged 1 commit into from Dec 4, 2023

Conversation

dlpierce
Copy link
Contributor

@dlpierce dlpierce commented Dec 1, 2023

Fixes

Fixes #6461

Summary

Move some code only useful when riiif is in use to the riiif initializer.
These files referenced code in the riiif gem which is only available in apps that have opted in.

Changes proposed in this pull request:

  • Move Hyrax::RiiifFile and Hyrax::RiiifFileResolver to the riiif.rb initializer
  • Update the generator template to match

@dlpierce dlpierce added the notes-bugfix Release Notes: Fixed a bug label Dec 1, 2023
@abelemlih abelemlih merged commit 94b3a1f into main Dec 4, 2023
4 checks passed
@abelemlih abelemlih deleted the mv-riiif-file branch December 4, 2023 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes-bugfix Release Notes: Fixed a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hyrax 5.0.0.rc2 depends on riiif
2 participants