Skip to content

[#1786] Adding documentation for Twig namespaced paths support #2211

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
merged 2 commits into from
Feb 8, 2013

Conversation

weaverryan
Copy link
Member

Hi guys!

See #1786 for details. Please let me know if you see any issues.

Q A
Doc fix? no
New docs? yes (symfony/symfony#5660)
Applies to 2.2
Fixed tickets #1786

Thanks!

Twig also natively offers a feature called "namespaced paths", and support
is built-in automatically for all of your bundles.


Copy link
Member

Choose a reason for hiding this comment

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

one empty line to much

@wouterj
Copy link
Member

wouterj commented Feb 3, 2013

I think you are one of the persons that should know that we like to see the pull request format in each PR message.. 😉

@weaverryan
Copy link
Member Author

Thanks @wouterj - I've fixed all your recommendations (all good ones!) and added the proper PR format (thanks for the reminder).

weaverryan added a commit that referenced this pull request Feb 8, 2013
[#1786] Adding documentation for Twig namespaced paths support
@weaverryan weaverryan merged commit 1409445 into master Feb 8, 2013
@weaverryan weaverryan deleted the twig-namespaced-paths branch February 8, 2013 18:33
@@ -376,6 +376,11 @@ When working with template inheritance, here are some tips to keep in mind:
Template Naming and Locations
-----------------------------

.. versionadded:: 2.2
Namespaced path support was added in 2.2, allowing for template names
like ``@AcmeDemoBundle/layout.html.twig``. See :doc:`/cookbook/templating/namespaced_paths`
Copy link
Member

Choose a reason for hiding this comment

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

should be @AcmeDemo

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