Skip to content

[Templating] added the private parameter in the TemplateController example #13104

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 1 commit into from
Feb 18, 2020

Conversation

zairigimad
Copy link
Contributor

No description provided.

Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

Thank you, just some minors comments

@@ -494,6 +497,8 @@ provided by Symfony:
<!-- special options defined by Symfony to set the page cache -->
<default key="maxAge">86400</default>
<default key="sharedAge">86400</default>
<!-- Whether or not caching should apply for client caches only -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- Whether or not caching should apply for client caches only -->
<!-- whether or not caching should apply for client caches only -->

@OskarStark
Copy link
Contributor

One question, is this a new feature introduced in 5.0 or was it just missing?

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

Ok to merge this in 4.4 since this part has been shorten, and we still have a dedicated note in 3.4.

@HeahDude HeahDude added this to the 4.4 milestone Feb 17, 2020
@HeahDude HeahDude changed the title Update templates.rst [Templating] added the private parameter in the TemplateController example Feb 17, 2020
@HeahDude HeahDude changed the base branch from 5.0 to 4.4 February 18, 2020 18:22
@HeahDude
Copy link
Contributor

Thank you! I've merged in 4.4 and applied the cs changes.

HeahDude added a commit that referenced this pull request Feb 18, 2020
…lateController example (zairigimad)

This PR was submitted for the 5.0 branch but it was squashed and merged into the 4.4 branch instead (closes #13104).

Discussion
----------

[Templating] added the `private` parameter in the TemplateController example

Commits
-------

9e4f7ab [Templating] added the `private` parameter in the TemplateController example
@HeahDude HeahDude merged commit 9e4f7ab into symfony:4.4 Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants