Skip to content

[Assetic] added a note about the need to remove a route #2220

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

Closed
wants to merge 2 commits into from
Closed

[Assetic] added a note about the need to remove a route #2220

wants to merge 2 commits into from

Conversation

bamarni
Copy link
Contributor

@bamarni bamarni commented Feb 6, 2013

_assetic route needs to be removed when use_controller is set to
false.

_assetic route needs to be removed when use_controller is set to
false.
@@ -337,6 +337,10 @@ the following change in your ``config_dev.yml`` file:
'use_controller' => false,
));

.. note::

You'll also have to remove the ```_assetic``` route in your ```app/config_dev.yml``` file.
Copy link
Member

Choose a reason for hiding this comment

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

You should indent this with 4 spaces and you must use 2 backtricks to indicate inline code blocks, not three.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok no pb

@bamarni
Copy link
Contributor Author

bamarni commented Feb 6, 2013

oh symfony/symfony-standard@16a98fd, I should compare my app configs against this repo more often...

@bamarni bamarni closed this Feb 6, 2013
@stof
Copy link
Member

stof commented Feb 11, 2013

@barmani your change is still right for 2.0. The automatic import of the Assetic resource is only done in the 2.1 version of the bundle

@bamarni bamarni mentioned this pull request Feb 11, 2013
weaverryan added a commit that referenced this pull request Feb 19, 2013
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