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 IiifPrint gem #2035

Closed
wants to merge 2 commits into from

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Nov 6, 2023

The log between the previous SHA of IiifPrint and the new one had the
following:

❯ git slog 9e7837ce4bd08bf8fff9126455d0e0e2602f6018..5fb1531c2aa4c4797aea5a83c74797148a9fbc32 --no-merges
* 4b4165f β€” 🎁 Add graceful fallback of preprocessing Jeremy Friesen, (2023-11-06) (origin/i282-gracefully-handling-of-copy-failure, i282-gracefully-handling-of-copy-failure)
* f8f9a90 β€” increase blacklight_iiif_search window Rob Kaufman, (2023-10-18) (origin/rails_version)
* 7fd53d6 β€” iiif_print doesnt use monads Rob Kaufman, (2023-10-17)
* b6352fa β€” loosen up hyrax too Rob Kaufman, (2023-10-17)
* a335c68 β€” Loosen up rails restriction Rob Kaufman, (2023-10-17)
* b170a57 β€” πŸ› Handle no file_set_ids property Jeremy Friesen, (2023-08-15) (adding-bug-fix)
* 7f34a12 β€” :broom: Update method call to new Shana Moore, (2023-08-22)
* 0af836f β€” πŸ› Fix calling Fedora from view Kirk Wang, (2023-08-22)
* e8e19df β€” Update app/views/hyrax/base/_representative_media.html.erb Jeremy Friesen, (2023-08-18)
* 95eed39 β€” πŸ› Render thumbnails when IIIF Print disabled Jeremy Friesen, (2023-08-18) (enable-thumbnails-when-iiif-print-disabled)

With adventist-knapsack running, I bashed into the web container and ran:

bundle update iiif_print derivative-rodeo --conservative

Yes the above derivative-rodeo is intentional instead of
derivative_rodeo which maps to the repository name; there are rubygems
issues to resolve.

Related to:

The log between the previous SHA of IiifPrint and the new one had the
following:

```
❯ git slog 9e7837ce4bd08bf8fff9126455d0e0e2602f6018..5fb1531c2aa4c4797aea5a83c74797148a9fbc32 --no-merges
* 4b4165f β€” 🎁 Add graceful fallback of preprocessing Jeremy Friesen, (2023-11-06) (origin/i282-gracefully-handling-of-copy-failure, i282-gracefully-handling-of-copy-failure)
* f8f9a90 β€” increase blacklight_iiif_search window Rob Kaufman, (2023-10-18) (origin/rails_version)
* 7fd53d6 β€” iiif_print doesnt use monads Rob Kaufman, (2023-10-17)
* b6352fa β€” loosen up hyrax too Rob Kaufman, (2023-10-17)
* a335c68 β€” Loosen up rails restriction Rob Kaufman, (2023-10-17)
* b170a57 β€” πŸ› Handle no file_set_ids property Jeremy Friesen, (2023-08-15) (adding-bug-fix)
* 7f34a12 β€” 🧹 Update method call to new Shana Moore, (2023-08-22)
* 0af836f β€” πŸ› Fix calling Fedora from view Kirk Wang, (2023-08-22)
* e8e19df β€” Update app/views/hyrax/base/_representative_media.html.erb Jeremy Friesen, (2023-08-18)
* 95eed39 β€” πŸ› Render thumbnails when IIIF Print disabled Jeremy Friesen, (2023-08-18) (enable-thumbnails-when-iiif-print-disabled)
```

With adventist-knapsack running, I bashed into the web container and ran:

```
bundle update iiif_print derivative-rodeo --conservative
```

Yes the above `derivative-rodeo` is intentional instead of
`derivative_rodeo` which maps to the repository name; there are rubygems
issues to resolve.

Related to:

- notch8/derivative_rodeo#70
jeremyf added a commit that referenced this pull request Nov 6, 2023
The log between the previous SHA of IiifPrint and the new one had the
following:

```
❯ git slog 9e7837ce4bd08bf8fff9126455d0e0e2602f6018..5fb1531c2aa4c4797aea5a83c74797148a9fbc32 --no-merges
* 4b4165f β€” 🎁 Add graceful fallback of preprocessing Jeremy Friesen, (2023-11-06) (origin/i282-gracefully-handling-of-copy-failure, i282-gracefully-handling-of-copy-failure)
* f8f9a90 β€” increase blacklight_iiif_search window Rob Kaufman, (2023-10-18) (origin/rails_version)
* 7fd53d6 β€” iiif_print doesnt use monads Rob Kaufman, (2023-10-17)
* b6352fa β€” loosen up hyrax too Rob Kaufman, (2023-10-17)
* a335c68 β€” Loosen up rails restriction Rob Kaufman, (2023-10-17)
* b170a57 β€” πŸ› Handle no file_set_ids property Jeremy Friesen, (2023-08-15) (adding-bug-fix)
* 7f34a12 β€” 🧹 Update method call to new Shana Moore, (2023-08-22)
* 0af836f β€” πŸ› Fix calling Fedora from view Kirk Wang, (2023-08-22)
* e8e19df β€” Update app/views/hyrax/base/_representative_media.html.erb Jeremy Friesen, (2023-08-18)
* 95eed39 β€” πŸ› Render thumbnails when IIIF Print disabled Jeremy Friesen, (2023-08-18) (enable-thumbnails-when-iiif-print-disabled)
```

With adventist-knapsack running, I bashed into the web container and ran:

```
bundle update iiif_print derivative-rodeo --conservative
```

Yes the above `derivative-rodeo` is intentional instead of
`derivative_rodeo` which maps to the repository name; there are rubygems
issues to resolve.

Related to:

- notch8/derivative_rodeo#70
- notch8/adventist-dl#643
- #2035
@jeremyf jeremyf added the patch-ver for release notes label Nov 6, 2023
@jeremyf jeremyf requested a review from kirkkwang November 6, 2023 20:59
The deec7e2 happened midflight with
6ccd618.  The former added a method
that the latter didn't know about.  The latter refactored to remove a
constant in favor of a class attribute.
@jeremyf jeremyf closed this Dec 21, 2023
@jeremyf jeremyf deleted the adventist-update-iiif-print branch December 21, 2023 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants