-
Notifications
You must be signed in to change notification settings - Fork 0
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
Drop ImageMagick and update submodules #15
base: master
Are you sure you want to change the base?
Conversation
@buccalon I've put it online at https://beta.images.imaginerio.org/s/imaginerio/page/welcome but the IIIF manifests are failing. Can you take a look at how things are configured in the admin panel? Ideally any changes you make would be able to be brought back into the code, but I understand that might not be a possibility. |
@davidheyman Thanks for setting this up. I broke the admin panel when upgrading the Clean URL module, and I can't access the logs. If you delete this module in the database (module column), it will work again. In my understanding, this is caused because the module is not handling the file
Of course, we could patch something in the docker setup, but this seems not adequate. @Daniel-KM can you confirm if we are doing something wrong here, please? Additionally, if I try to locally upload a file I run into the permission issue that affects files created by the docker user: @davidheyman Let me know if there is anything I can set up in my environment to fix this problem. I read about matching the user ID with the Docker ID, but I hope there is a better way to handle volume permissions in the docker-compose file. Currently I use chmod in the files directory everytime I clone the repo. |
For CleanUrl, the copy of the original file is required to bypass the original routes when the main slug is removed. So it should be managed by the box. |
Issue: #14
@davidheyman let me know if you want to test this against our database before deploying it