Skip to content
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

Adding :requirements to routes so it's possible to use periods etc in path #159

Merged
merged 1 commit into from
Apr 4, 2012

Conversation

gaiottino
Copy link
Contributor

This is a pull request that simplifies the work done by @rodzyn here: #145

The main differences is that it's updated for the latest master, and also doesn't change the handling of the default format. The difference from how things work today is that it doesn't throw an exception if the :format is not part of the allowed content_types.

@phlegx
Copy link

phlegx commented Apr 4, 2012

1+

mbleigh pushed a commit that referenced this pull request Apr 4, 2012
Adding :requirements to routes so it's possible to use periods etc in path
@mbleigh mbleigh merged commit 8ee8481 into ruby-grape:master Apr 4, 2012
@rodzyn
Copy link
Contributor

rodzyn commented Apr 6, 2012

Finally! ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants