Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

Symfony 4.4 & 5 compatibility #216

Open
cbeyer opened this issue Mar 10, 2021 · 5 comments
Open

Symfony 4.4 & 5 compatibility #216

cbeyer opened this issue Mar 10, 2021 · 5 comments

Comments

@cbeyer
Copy link
Contributor

cbeyer commented Mar 10, 2021

Hi,

I have upgraded my project to Symfony 4.4 and now my timelines don't show anyhting.

composer.json

        "predis/predis": "^1.1",
        "snc/redis-bundle": "^3.3",
        "stephpy/timeline-bundle": "^3.1",

spy_timeline.yaml

spy_timeline:
    render:
        path:     'Timeline/Timeline'
        fallback: 'Timeline/Timeline/default.html.twig'

I assume the bug comes from the following line

return vsprintf('%s:%s.html.twig', array(

Replace with '%s/%s.html.twig' ?

@stephpy
Copy link
Owner

stephpy commented Mar 25, 2021

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:

  • You define template into twig functions
  • We create a new version of bundle for this change, looks a bit overkill

@cbeyer
Copy link
Contributor Author

cbeyer commented Apr 6, 2021

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.

@cbeyer cbeyer changed the title Symfony 4.4 compatibility Symfony 4.4 & 5 compatibility Jun 4, 2022
@cbeyer
Copy link
Contributor Author

cbeyer commented Jun 4, 2022

Hi, now I upgrade my project to SF5
It seems this bundle is not compatible :(
https://packagist.org/packages/stephpy/timeline-bundle

Any workaround ?

@cbeyer
Copy link
Contributor Author

cbeyer commented Jun 6, 2022

Nobody ? :(

@stephpy
Copy link
Owner

stephpy commented Jul 25, 2022

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants