-
Notifications
You must be signed in to change notification settings - Fork 2
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 to use new starter site #10
Conversation
@nigelgbanks I think the build will fail because of Drush Launcher and Drush 12. That's what seemed to happen on my machine (the drupal container balked and exited). I put it here hoping more eyeballs would see it but it seems that PRs don't trigger a build anymore. |
I tried locally, and made a few changes to your pull. Mainly, I changed the source images for ISLANDORA_TAG=2.0.10 And a change to Followed by: docker compose build
docker compose pull Seems to work. |
With your fixes, it runs! 🥳 I encountered an error w.r.t. IIIF manifest generation that Born Digital has a fix for (thank you!) and I think it'll be worth waiting till the books display in Mirador properly before launching this. Thus, I'm keeping it in draft for now. |
I saw a few conflicts with the IIIF module. |
I bumped the version to 3.0.7. Mirador books now display properly. |
Tested locally and this looks to work well. |
Updates the hashes and commits of Workbench, Demo Objects, and Starter Site.