Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

πŸ› Update IiifPrint gem #643

Merged
merged 1 commit into from
Nov 6, 2023

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)

I'm also including the docker-compose.bundle.yml file. With the
application spun down I edited the Gemfile then ran:

docker compose -f ./docker-compose.yml -f ./docker-compose.bundle.yml up

That bundled the gems. To get the updated logging in the Derivative
Rodeo, I then shelled into the web container and ran:

bundle update 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)
```

I'm also including the `docker-compose.bundle.yml` file.  With the
application spun down I edited the `Gemfile` then ran:

```bash
docker compose -f ./docker-compose.yml -f ./docker-compose.bundle.yml up
```

That bundled the gems.  To get the updated logging in the Derivative
Rodeo, I then shelled into the web container and ran:

```
bundle update 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 to samvera/hyku 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
@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Nov 6, 2023

Rob just submitted this PR. Could we merge it and run update again? @jeremyf

@ShanaLMoore ShanaLMoore merged commit c7e2854 into main Nov 6, 2023
@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Nov 6, 2023

omg, I accidentally just merged this branch after requesting changesπŸ™ƒ we'll need another PR just like this. Sorry! πŸ˜“ @jeremyf

@jeremyf jeremyf deleted the update-iiif-print-and-derivative-rodeo branch December 1, 2023 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants