[Book][Translation] Added tip for routing params #4803
Conversation
xelaris
commented
Jan 9, 2015
xelaris
commented
| Q | A |
|---|---|
| Doc fix? | no |
| New docs? | no |
| Applies to | 2.3+ |
| Fixed tickets |
|
👍 |
| .. tip:: | ||
|
|
||
| Read :doc:`/cookbook/routing/service_container_parameters` to learn, how to | ||
| avoid hardcoding the ``_locale`` requirement in all your routes. |
There was a problem hiding this comment.
I have a question about the use of the word requirement in this tip. Are you referring to a real route requirement option or are you using it as a regular word? If that's the case, we could change it to avoid confusions.
There was a problem hiding this comment.
I took it from the referenced entry
You could hardcode your
_localerequirement in all your routes [...]
For me it's referring to the route option but I understand your question, it is maybe also interpretable as the regular word. As a non-native speaker I would say, the sentence is still valid then, isn't it?
There was a problem hiding this comment.
This is exactly about route requirements as in the example route above, isn't it?
|
@wouterj I removed the comma in the new tip and in another one for consistency. |
|
Nice addition - thanks Alexander! |
This PR was merged into the 2.3 branch. Discussion ---------- [Book][Translation] Added tip for routing params | Q | A | ------------- | --- | Doc fix? | no | New docs? | no | Applies to | 2.3+ | Fixed tickets | Commits ------- acb5868 [Book][Translation] Added tip for routing params
