Skip to content

Update routing.rst #4844

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update routing.rst #4844

wants to merge 1 commit into from

Conversation

bglamer
Copy link
Contributor

@bglamer bglamer commented Jan 15, 2015

The first code type of "requirements of route conditions" is marked as PHP but it is annotations.

@bglamer
Update routing.rst
ed883c4 
The first code type of "requirements of route conditions" is marked as PHP but it is annotations.
@xabbuh
Copy link
Member

xabbuh commented Jan 15, 2015

Great catch @bglamer! 👍

@weaverryan @wouterj Should be merged into the 2.3 branch.

@wouterj
Copy link
Member

wouterj commented Jan 16, 2015

👍 Thanks!

wouterj added a commit that referenced this pull request Jan 16, 2015
@wouterj
minor #4844 Update routing.rst (bglamer)
9680ec0 
This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #4844).

Discussion
----------

Update routing.rst

The first code type of "requirements of route conditions" is marked as PHP but it is annotations.

Commits
-------

313c1b9 Update routing.rst
@wouterj wouterj closed this Jan 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bglamer @xabbuh @wouterj