Skip to content

[Fwb][TwigBundle] Remove Twig Preview controller routing in favor of fwb #675

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

Merged
1 commit merged into from
Nov 14, 2019

Conversation

ogizanagi
Copy link
Contributor

Q A
License MIT
Doc issue/PR N/A

AFAIU, the controller for this route is deprecated in favor of the HttpKernel one registered through the fwb recipe. So new projects starting with Twig 4.4 should not start with this deprecated route, and use the fwb one instead.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Pull request passes validation.

ghost pushed a commit that referenced this pull request Nov 14, 2019
@ghost ghost merged commit e93e95b into symfony:master Nov 14, 2019
@ogizanagi ogizanagi deleted the rm_twig_preview_controller branch November 14, 2019 10:11
javiereguiluz added a commit to symfony/symfony-docs that referenced this pull request Jun 4, 2020
…alteschlueter)

This PR was merged into the 4.4 branch.

Discussion
----------

Update information in flex about added files for twig

The information about the recipe files of the twig-bundle are outdated for symfony 4.4 and 5.

* The route configuration file for the dev environment in the recipe of the twig-bundle was removed last year
  * symfony/recipes#675
* A package configuration file was added for the test environment
  * symfony/recipes#666
  * Maybe this change must be also documented in 3.4 or what do you think?

Commits
-------

91ee76e Update information in flex about added files for twig
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants