Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Documented HTTP caching and user sessions#9667

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@Toflar
Copy link
Contributor

As promised in#9647.

@javiereguiluz
Copy link
Member

javiereguiluz commentedApr 23, 2018
edited
Loading

@Toflar thanks a lot for this contribution! It's perfectly explained and it's easy to follow.

However, I made some rewords to make the explanations more concise. We do this routinely not only for the readers, but also for us, to make contents easier to maintain. Thanks!

@Toflar
Copy link
ContributorAuthor

It's perfectly fine. Rewording by the same people makes sure the readers have a consistent reading experience, it's perfect! Thank you very much!

@Toflar
Copy link
ContributorAuthor

BTW: What's the policy to use.. versionadded:: 4.1? Should we maybe also use that here?

@javiereguiluz
Copy link
Member

We include a.. versionadded:: for all new features and also for deprecations. And yes, we should use it here and that's why I included at the end, just after the code example that uses the new feature 😄

@Toflar
Copy link
ContributorAuthor

Oh man, what a morning. I miss everything 🤣 Sorry, Javier! And thanks for the patience 😃

@javiereguiluz
Copy link
Member

And it's merged!! Thanks Yanick!

@javiereguiluzjaviereguiluz merged commitecb0ac4 intosymfony:masterApr 24, 2018
javiereguiluz added a commit that referenced this pull requestApr 24, 2018
…eguiluz)This PR was merged into the master branch.Discussion----------Documented HTTP caching and user sessionsAs promised in#9647.Commits-------ecb0ac4 Minor reword to make it more concise9b37b0e Documented HTTP caching and user sessions
@ToflarToflar deleted the http-caching-and-sessions branchApril 24, 2018 08:01
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

4.1

Development

Successfully merging this pull request may close these issues.

3 participants

@Toflar@javiereguiluz@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp