Update page_creation.rst #3572
Update page_creation.rst #3572
Conversation
| @@ -70,7 +70,7 @@ by going to the following URL: | |||
|
|
|||
| .. code-block:: text | |||
|
|
|||
| http://localhost/app_dev.php/hello/Symfony | |||
| http://localhost/app_dev.php/demo/hello/Symfony | |||
There was a problem hiding this comment.
That's not right. The route path is /hello/{name} not /demo/hello/{name}.
There was a problem hiding this comment.
@mjankiewicz is correct, the demo routes are prefixed with /demo.
|
Are you sure ? :) I'm reading now documentation and prepare application (with AcmeBundle). In my case an link to "hello..." is : http://sf2.loc/app_dev.php/demo/hello/World (Acme Bundle) but maybe I think wrong. |
|
Nice one, @mjankiewicz ! This error is here for a really long time and you are the first one who spotted it (well, at last the first one who wanted to fix it!). Thank you! Btw, you don't have to create a new PR, but fixes should be based on the 2.3 branch and not the 2.4 branch. This is an inconvinience of the doc "edit" button... |
|
Well, do we refer to the standard edition or to the example in this chapter? If we want to be consistent with the AcmeDemoBundle from the standard edition, the routing example also needs to be updated. |
|
Hmm, indeed. Even I read the docs wrong. @mjankiewicz that chapter is creating a new controller which is bound to |
No description provided.