Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update http_cache.rst #4809

Closed
wants to merge 1 commit into from
Closed

Update http_cache.rst #4809

wants to merge 1 commit into from

Conversation

ifdattic
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to 2.3
Fixed tickets

@ifdattic
Update http_cache.rst
96d5e9d 
| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3
| Fixed tickets |
ifdattic
ifdattic reviewed Jan 11, 2015
View reviewed changes
book/http_cache.rst
@@ -856,7 +856,7 @@ If one content corresponds to one URL, the ``PURGE`` model works well.
You send a request to the cache proxy with the HTTP method ``PURGE`` (using
the word "PURGE" is a convention, technically this can be any string) instead
of ``GET`` and make the cache proxy detect this and remove the data from the
cache instead of going to Symfony to get a response.
cache instead of going to the application to get a response.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The topic mostly uses application and does tie itself to Symfony, as this part is more about general working of cache I think it's a good idea to also just use application

@ifdattic
Copy link
Contributor Author

Line 1045, the PHP example doesn't have a great readability and in my opinion could use some improvements

@wouterj
Copy link
Member

wouterj commented Jan 11, 2015

Please note #4779 , where I've updated the examples.

@wouterj
Copy link
Member

wouterj commented Jan 16, 2015

Closing in favor of #4861

@wouterj wouterj closed this Jan 16, 2015
@ifdattic ifdattic deleted the patch-7 branch January 17, 2015 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ifdattic @wouterj