Skip to content

Forward slash regex alternative #7973

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

Closed
wants to merge 1 commit into from
Closed

Conversation

webcookie
Copy link

When using the _format parameter in a route with the existing regex to allow slashes, it will append the _format onto your parameter value. Providing an alternative regex for this scenario.

When using the _format parameter in a route with the existing regex to allow slashes, it will append the _format onto your parameter value. Providing an alternative regex for this scenario.
@javiereguiluz
Copy link
Member

@webcookie thanks for this contribution and I'm sorry we didn't review it on time. I like it, but I can't merge it because of the conflicts. Besides, I prefer to add it as a note instead of a new section, so I've opened #9165 to finish this. Thanks!

javiereguiluz added a commit that referenced this pull request Jan 30, 2018
…param (javiereguiluz)

This PR was merged into the 2.7 branch.

Discussion
----------

Added a note about using slashes and the special _format param

This replaces #7973.

Commits
-------

fb9d377 Added a note about using slashes and the special _format param
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.

3 participants