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

Added cookbook entry about FrameworkBundle:TemplateController#2002

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

Closed
kingcrunch wants to merge1 commit intosymfony:masterfromkingcrunch:feature/static-template-cache
Closed

Added cookbook entry about FrameworkBundle:TemplateController#2002

kingcrunch wants to merge1 commit intosymfony:masterfromkingcrunch:feature/static-template-cache

Conversation

@kingcrunch
Copy link
Contributor

Seesymfony/symfony#6083

Added docbook entry aboutFrameworkBundle:TemplateController. I hope, that's ok so far.

Copy link
Member

Choose a reason for hiding this comment

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

Use double backtricks to indicate inline code.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

No problem, but what is a single backtick used for? Found it somewhere else (cookbook/templating/twig_extension.rst) and assumed, that they were used for keywords.

Copy link
Member

Choose a reason for hiding this comment

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

They are used for hyperlinks:

This is some text with `A hyperlink`_ and you can refer to a doc article: :doc:`/cookbook/templating/twig_extension` .. _`A hyperlink`: http://symfony.com/

@stof
Copy link
Member

stof commentedDec 5, 2012

@weaverryan The support of caching is new in 2.2 but the controller itself is a 2.0 feature

Copy link
Member

Choose a reason for hiding this comment

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

there is a missing: here.

@kingcrunch
Copy link
ContributorAuthor

Fix the missing colon, rebased and squashed. Hopefully now? :X

weaverryan added a commit that referenced this pull requestDec 26, 2012
…atures and proofreadingThe caching stuff will be added back on the master branch
weaverryan added a commit that referenced this pull requestDec 26, 2012
@weaverryan
Copy link
Member

Hi Sebastian!

Thanks for the PR - I patched it originally into the 2.0 branch at sha:8cbd129, since this is documenting a feature that's been available since 2.0, but wasn't documented yet (and I'm very happy to have it documented now!). I then removed the 2.2-only features at sha:fa6fdee, merged up to master, then re-added those to master only at sha:42b698d. Additionally, things needed to be updated now for the change to howrender works since the security update.

If you see any issues at all, please let me know!

Thanks!

@stofstof mentioned this pull requestDec 28, 2012
Sign up for freeto 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.

5 participants

@kingcrunch@stof@weaverryan@fabpot@wouterj

[8]ページ先頭

©2009-2025 Movatter.jp