-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add the data_uri filter #3102
Add the data_uri filter #3102
Conversation
bc41afd
to
f2dac7a
Compare
This PR was merged into the 2.x branch. Discussion ---------- Add the data_uri filter Implemented as a popular request here: closes twigphp/Twig-extensions#54 Commits ------- 220e750 added the data_uri filter
@@ -24,13 +24,14 @@ | |||
} | |||
], | |||
"require": { | |||
"php": "^7.0", | |||
"php": "^7.1.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be careful here: Symfony 3.4 would e.g. bump to "twig/twig": "^1.43|^2.12"
, people on php 7.0 would be forced to downgrade to Twig 1 in order to continue receiving Symfony upgrades.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have one client who will be locked on Twig 2.11 now. I don't want to complain becuase nobody really should be on php 7.0 anymore these days. (Also, I urge them to upgrade php everytime we talk)
Anyway, if that change is kept, we should at least document the bump in the ChangeLog.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see #3157
Implemented as a popular request here: closes twigphp/Twig-extensions#54