Improve Quick Tour "Big Picture" section and expand Environments docs #2823
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
Contributor
jbafford
commented
Jul 12, 2013
| Q | A |
|---|---|
| Doc fix? | yes |
| New docs? | no |
| Applies to | 2.2+ |
| Fixed tickets | #2672 |
- Add a description of environments and front controllers to book/page_creation
- Improve description of the profiler
- Improve the Working with Environments section
- Addresses comments from PR Reorganize and expand quickstart big picture docs on environments #2683 and targets the 2.2 docs instead of 2.0
* Add Environments section under "Understanding the Fundamentals" to introduce environments. This uses the glossary definition as a starting point and expands on it a little. * Move the Web Profiler discussion out of Working with Environments into a separate section, and add a paragraph briefly summarizing what the profiler can do. * Improve the Working with Environments section. * Original changes from PR symfony#2683 rebased on 2.2
* Move the environments and front controller discussions to book/page_creation * This allows for the debug toolbar discussion to be more prominent
|
|
||
| To make your application respond faster, Symfony2 maintains a cache under the | ||
| ``app/cache/`` directory. In the ``dev`` environment, this cache is flushed | ||
| automatically whenever you make changes to any code or configuration. But that's |
There was a problem hiding this comment.
not true. Refreshing it automatically is not based on the environment but on the debug mode. The environment and the debug mode are independant (see the signature of the Kernel constructor)
|
@stof thanks for the comments. I've updated the PR with fixes that should cover them. |
weaverryan
added a commit
that referenced
this pull request
Jul 20, 2013
Improve Quick Tour "Big Picture" section and expand Environments docs
weaverryan
added a commit
that referenced
this pull request
Jul 20, 2013
* Making things shorter where possible * Being more consistent in web server setup and URLs * Improving language, wording, etc
|
Hi John! Thanks for taking this on - this document has grown and changed over time, so we have some issues. I've taken your changes and made even more at sha: 72072e9. If you see any issues, let me know. Thanks! |
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.