-
Notifications
You must be signed in to change notification settings - Fork 9
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
formatOptions is not applied on base image #15
Comments
Hello, is it something that works properly in Payload? |
Not quite correct: Payload does adjust the base image a bit. Moreover, you can pass payload-s3-upload/src/buildUploadHook.ts Lines 30 to 34 in 3e0417c
The result is not included in Maybe the data object could give a general idea about the content contained in the edited file. payload-s3-upload/src/buildUploadHook.ts Lines 11 to 14 in 3e0417c
|
I was talking about the cloud storage plugin of Payload, sorry if it was misleading. The problem I see is that the As a result plugins can't access the transformed image buffer. It's a bug from Payload imo, I see no reason not to pass the transformed buffer to the plugin chain. |
I'm wrong Payload does pass the transformed image buffer downstream. What should be done differently in here?: payload-s3-upload/src/buildUploadHook.ts Lines 15 to 38 in 3e0417c
We just get data passed by Payload after transformations are applied. |
I think payload does not expose the result file buffer directly, it is included in someway, but not correctly referenced. Line 33 ignores the mime type of the created file size completely I think But if I'm not quite sure what payload is exactly giving to the plugins. I'm just trying to give an initial idea of what could be the cause. |
When uploading image files, the formatOptions are only applied to "imageSizes" and not to the base image.
The text was updated successfully, but these errors were encountered: