michaelmcandrew / civicrm-buildkit-docker

This is a read only copy. Please make PRs here: https://lab.civicrm.org/michaelmcandrew/civicrm-buildkit-docker
https://lab.civicrm.org/michaelmcandrew/civicrm-buildkit-docker
GNU Affero General Public License v3.0
40 stars 31 forks source link

Enable imagick #30

Closed wmortada closed 5 years ago

wmortada commented 5 years ago

I came across an issue getting Mosaico to work with CiviCRM. I got an error that the imagick extension was missing. Looking at the dockerfile, I can see that the php imagick module is installed but not enabled. Adding this line will enable this module.

I presume that you will need to run your publishing script to update all of the docker files. Unfortunately, I'm unable to do this on my machine at the moment.

michaelmcandrew commented 5 years ago

Thanks a lot @wmortada - I've made the changes and published a new image on docker hub.

I took a slightly different route https://github.com/michaelmcandrew/civicrm-buildkit-docker/commit/1c3881f1cb5e38a458d9f0ea0dc6c13352613373 and merged that, which means your commit will get missed from the your PR - apologies for that - I should have built on your PR to write you into the history - will do so next time...

michaelmcandrew commented 5 years ago

Nice spot in any case - great to get be able to run mosaico out of the box with this

wmortada commented 5 years ago

No worries! Glad to help out.

Yes, it is great to be able to run Mosaico out of the box like this. I needed it for a demo at my CiviCRM meet up on Wednesday so glad to get that working!