Skip to content
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

Wp amazon s3 and cloudfront #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

noreabu
Copy link

@noreabu noreabu commented Apr 19, 2016

Rewrote some calls, so that the necessary hooks from the plugin wp-amazon-s3-and-cloudfront are triggered.

I think there is still room for improvement and i'm happy for feedback.

demmel added 2 commits April 19, 2016 10:42
rewrote/added some calls to trigger the neccessary hooks of plugin: wp-amazon-s3-and-cloudfront
images were overwritten by wp_generate_attachment_metadata
@GarethAlinedCreativeDesign

+1

Would love this in main, we've disabled Amazon S3 Plugin in the meantime.

@noreabu
Copy link
Author

noreabu commented May 17, 2016

We still use Amazon S3 Plugin and the merge is included in a docker build process. I can write down the details if you are interested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants