silverstripe / silverstripe-s3

Silverstripe module to store assets in S3 rather than on the local filesystem (SS4/SS5 only)
BSD 3-Clause "New" or "Revised" License
20 stars 25 forks source link

Sends full-size images as thumbnails #14

Closed chillu closed 5 years ago

chillu commented 6 years ago

Excerpt of a GraphQL response in admin/assets

silverstripe - files 2017-10-10 14-12-45

obj63mc commented 5 years ago

Hi @chillu - not sure if this is still an issue. Testing on our production site and this module with #20 merged I am getting back the following -

Screen Shot 2019-04-08 at 8 57 43 AM

Looks like it is returning back specific 'Fit width' image files. If you still see this issue let me know and I can look into it some more.

madmatt commented 5 years ago

@chillu I suspect this might be resolved in 0.3.0 or 0.4.0, if you get a chance to re-test could you let us know?

If not, I think we can consider this fixed with @obj63mc's recent changes.