-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Improve example context #7563
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
Improve example context #7563
Conversation
This commit add the component standalone examples and place the existing ones into a controller context.
Improve example context
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
{ | ||
public function newAction() | ||
{ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This blank line is useless.
$form = $this->createFormBuilder($task) | ||
->setAction($this->generateUrl('target_route')) | ||
->setMethod('GET') | ||
->add('task', TextType::class) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR should target 2.7 and use type names (i.e 'text'
).
Thank you @matthieu88160. |
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #7563). Discussion ---------- Improve example context This commit add the component standalone examples and place the existing ones into a controller context. (Framework example first) Commits ------- 901fe20 Fixed the RST syntax ce9db18 Improve example context
This commit add the component standalone examples and place the existing ones into a controller context. (Framework example first)