[Routing] Added a note about defining route requirements in the container #1827
Conversation
| @@ -612,6 +612,16 @@ the regular expression ``(en|fr)``. | |||
| | /es | *won't match this route* | | |||
| +-----+--------------------------+ | |||
|
|
|||
| .. sidebar:: Requirements defined in the container | |||
|
|
|||
| If for some reason you need to define some configurable requirements, | |||
There was a problem hiding this comment.
I'm not good in English grammar, but I think this should be:
If, for some reason, you need to define some configurable requirements,
There was a problem hiding this comment.
Me neither, I'm Cuban and I speak Spanish. Probably you are right. Thanks.
|
@wouterj What do you think now? |
|
I think this should be a cookbook article, to be easier to find (as it would be linked from the index page) |
|
@stof Don't you think is to little content for a cookbook article? If not, the I'll be glad to do it. |
| .. code-block:: yaml | ||
|
|
||
| contact: | ||
| pattern: /_locale/contact |
There was a problem hiding this comment.
should be /{_locale}/contact (same for xml and php format)
There was a problem hiding this comment.
Sorry, it was a typo. Fixed!
|
This feature is only available since 2.1. So this PR should be done on the 2.1 branch. |
|
Please also mention that you can use these kernel params in the pattern for static text like |
|
@Tobion The make this PR againt 2.1 branch, should I close this and reopen again, or there is another way? |
|
You need to open a new pr. |
|
Ok. Thanks for the tip! 2012/10/19 Tobias Schultze notifications@github.com
|
|
See #1832 |
Yesterday I was searching how to do this and I can't find nothing in the docs. Thanks @Tobion