Skip to content

Documented the inline requirements and default values for routes #9520

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

Merged
merged 1 commit into from
Apr 3, 2018

Conversation

javiereguiluz
Copy link
Member

This fixes #9459.

routing.rst Outdated
xsi:schemaLocation="http://symfony.com/schema/routing
http://symfony.com/schema/routing/routing-1.0.xsd">

<route id="blog_list" path="/blog/{page <\d+>}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it normal to have here a space before < ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch! That was a bug. Fixed now. Thanks!

@javiereguiluz javiereguiluz merged commit a3b8666 into symfony:master Apr 3, 2018
javiereguiluz added a commit that referenced this pull request Apr 3, 2018
… routes (javiereguiluz)

This PR was squashed before being merged into the master branch (closes #9520).

Discussion
----------

Documented the inline requirements and default values for routes

This fixes #9459.

Commits
-------

a3b8666 Documented the inline requirements and default values for routes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Routing] Allow inline definition of requirements and defaults
3 participants