- π¨π¦Canada bbombachini London, ON
This is great idea and it works, but I'm moving this to "Needs work" because we need a config form and be able to import/export config as well as reflect this change on schema.
- πΊπΈUnited States joshf
I took a different approach for this. Instead of a configuration form, I opted to use the default file scheme as suggested by Damien in the D7 version of this issue. β . This works great for us with our s3 setup, but there is one weird caveat.
In 2715669 β , the thumbnail URL was changed to relative to keep content portable between hosts. Unfortunately this doesn't work with s3 because the normal operation is that the image is on a different host from Drupal. I've tried to work around this by first checking if the hostname of the preview image is the same as the Drupal hostname. If it is, the existing behavior is retained and the image is output as relative. If the hostnames differ, the full path including original host is used.
The only time I can see this being an issue is if someone is staging their content on a server that uses a different s3 server than production, which sounds super complicated. In that situation, preview images are already broken, so this doesn't break anything new but does fix s3 preview images for those of us who aren't staging their content.
I'm leaving #2's patch visible since it's a completely different way of going about it that might work better for some.
- Status changed to Needs review
about 1 year ago 9:24pm 14 November 2023