-
-
Notifications
You must be signed in to change notification settings - Fork 35
Conversation
There is no usage in Twig templates too? |
You're right 😟 |
5e089bd
to
f5be31d
Compare
src/Resources/config/twig.xml
Outdated
<?xml version="1.0" encoding="UTF-8"?> | ||
<container xmlns="http://symfony.com/schema/dic/services" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd"> | ||
<services> | ||
<service id="sonata.timeline.twig.intl_extension" class="Twig\Extensions\IntlExtension"> |
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.
This is needed?
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.
When I've used them in a project, I had to add them as services. In the docs it says:
use Twig\Extra\Intl\IntlExtension;
$twig = new \Twig\Environment(...);
$twig->addExtension(new IntlExtension());
which I haven't check but I guess is what twig.extension
tag does.
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.
Okay then. We could keep this for the stable release to stay BC. We should remove this service and update the installation instruction for the next major release.
f5be31d
to
4c4503e
Compare
4c4503e
to
d436de5
Compare
Could you please rebase your PR and fix merge conflicts? |
d436de5
to
76f3706
Compare
Since the minimum is now |
Ups |
Thanks @franmomu ! |
Subject
Ref sonata-project/SonataIntlBundle#280
I am targeting this branch, because this is BC.
Changelog