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

Update LiipImagineBundle #847

Closed
wants to merge 2 commits into from

Conversation

pepamartinec
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets none

Update legacy LiipImagineBundle 0.20.2 to 1.0+.

We use this with version 1.3.1 in production for about 2 weeks and no problem occured so far.

@roderik
Copy link
Contributor

roderik commented Nov 29, 2015

Until there is a good solution for liip/LiipImagineBundle#584, we cannot upgrade

@pepamartinec
Copy link
Contributor Author

Oh, I hit this problem too but I considered it to be my fail because I allowed user to pick an PNG image where JPG was expected by the filter.

I'm closing this PR for now.

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