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

DO NOT MERGE, FOR QA ONLY! EZP-29749: As an Administrator I want to configure Imagine ProxyResolver #2506

Closed
wants to merge 3 commits into from

Conversation

kmadejski
Copy link
Member

Question Answer
JIRA issue EZP-29749
Bug/Improvement yes
New feature yes (?)
Target version 6.7/6.13/2.3/master
BC breaks no
Doc needed yes

@kmadejski kmadejski self-assigned this Dec 19, 2018
@kmadejski kmadejski changed the base branch from master to 7.3 December 19, 2018 13:50
@micszo micszo self-assigned this Dec 19, 2018
@micszo
Copy link
Member

micszo commented Dec 20, 2018

Test OK on 2.3 and 2.4.
Thanks for the PR.

@micszo micszo removed their assignment Dec 20, 2018
@micszo micszo closed this Dec 20, 2018
@@ -1,5 +1,8 @@
parameters:
liip_imagine.filter.configuration.class: eZ\Bundle\EzPublishCoreBundle\Imagine\Filter\FilterConfiguration
liip_imagine.cache_resolver.proxy.class: Liip\ImagineBundle\Imagine\Cache\Resolver\ProxyResolver
Copy link
Contributor

@andrerom andrerom Dec 20, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not how we role on Symfony 3x ;)

ref: 4896acf#diff-5f3dc6c12e2a442a9d234416275f9555

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

Successfully merging this pull request may close these issues.

4 participants