-
Notifications
You must be signed in to change notification settings - Fork 57
Symfony 4.4 & 5 compatibility #216
Comments
Hi, I guess it's a BC break change, since %s/%s.html.twig is may not supported in previous version of Symfony/Twig, can you check if it's BC break ? If it's BC break I see 2 solutions:
|
I am not expert enough to say if it is a BC or not ;) but while upgrading to SF4.4 I have changed multiple Twig calls, replacing ":" by "/". Here you have an example of an upgrading article https://codereviewvideos.com/course/upgrade-symfony-3-to-symfony-4/video/part-2-of-2-migrating-to-symfony-4-with-flex (search "This needs the include syntax updating") Il my opinion your current code doesn't work with SF4.4. |
Hi, now I upgrade my project to SF5 Any workaround ? |
Nobody ? :( |
Hello, Sorry but i cannot find time to support this project :( You can by this way fork it then add your own supports to sf 4.x, 5.x. Sorry about that, have a good luck. |
Hi,
I have upgraded my project to Symfony 4.4 and now my timelines don't show anyhting.
composer.json
spy_timeline.yaml
I assume the bug comes from the following line
timeline-bundle/Twig/Extension/TimelineExtension.php
Line 299 in 207adaf
Replace with
'%s/%s.html.twig'
?The text was updated successfully, but these errors were encountered: