Explain how to override the default templates directory #7844
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
linaori
reviewed
Apr 27, 2017
|
|
||
| If your templates are not stored in the default ``app/Resources/views/`` | ||
| directory, use the :ref:`twig.paths <config-twig-paths>` configuration option to | ||
| define your own templates directory (or directories): |
There was a problem hiding this comment.
What about mentioning the root dir here? Either via a note or %kernel.root_dir%/Resources/views/ instead of app/Resources/views/. This will be one of the things people will forget to change once they move their kernel to src/. People already are in the process of doing so now that the structure for 4.0 has been presented.
There was a problem hiding this comment.
What about mentioning the root dir here? Either via a note or %kernel.root_dir%/Resources/views/ instead of app/Resources/views/.
I agree we should explicit it at some point but I don 't think this is the right place to do so, since other resources are concerned (i.e translations).
The whole Resources folder should be relative to the kernel root dir.
|
👍 |
|
Thank you Javier. |
xabbuh
added a commit
that referenced
this pull request
May 11, 2017
xabbuh
added a commit
that referenced
this pull request
May 11, 2017
* 2.7: [#7844] add XML and PHP config examples Property access fix remaining setDefinition() call mention old way to create PHPUnit mock objects Explain how to override the default templates directory Added a new article about linting translation files Removed any reference to X-Status-Code
xabbuh
added a commit
that referenced
this pull request
May 11, 2017
* 2.8: [#7844] add XML and PHP config examples Property access fix remaining setDefinition() call mention old way to create PHPUnit mock objects Explain how to override the default templates directory Added a new article about linting translation files Removed any reference to X-Status-Code
xabbuh
added a commit
that referenced
this pull request
May 11, 2017
* 3.2: Documented the getIterator() method [#7844] add XML and PHP config examples Property access fix remaining setDefinition() call mention old way to create PHPUnit mock objects Explain how to override the default templates directory Added a new article about linting translation files Removed any reference to X-Status-Code
weaverryan
added a commit
that referenced
this pull request
May 20, 2017
* origin/2.7: (51 commits) Avoid backticks in shell scripts Update optional_dependencies.rst Fix xml blocks [#7875] minor tweaks Minor fix Minor changes [#7773] fix line length Add helpful remarks on custom DataCollector Remove use of deprecated security.exception_listener.class parameter Update resources.rst Fix incoherent ut8mb4 collation in Doctrine setup Fix decorating service definition fix empty XML element [#7873] add missing toctree entries Reworded the introduction of the contributing docs [#7844] add XML and PHP config examples Property access [HttpFoundation] Fix clearstatcache first arg Explain how to provide a stack trace fix remaining setDefinition() call ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes #4714.