search

jrrdnx / craft-cloudflare-r2

MIT License
6 stars 3 forks source link

Remove focal point detection #2

Closed lenvanessen closed 1 year ago

lenvanessen commented 1 year ago

Focal points are not supported by Cloudflare R2, so this method would probably be unnecessary

jrrdnx commented 1 year ago

Thanks for this! I was leaving this plugin, which is essentially a fork of Craft's own S3 plugin, as close as possible to the original so I could keep it up to date with any changes. There were also several methods not being used but that are still required since this is extending craft\flysystem\base\FlysystemFs. That doesn't appear to be the case with detectFocalPoint(), but I think I'll keep this in for now since it's not doing any harm.

lenvanessen commented 1 year ago

Makes sense!