Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.