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

Fix param with period not work #47

Closed
wants to merge 1 commit into from
Closed

Fix param with period not work #47

wants to merge 1 commit into from

Conversation

ningt
Copy link

@ningt ningt commented Jul 9, 2015

Refer to Issue 5

@jacwright

@ningt ningt closed this Aug 6, 2015
@baskettcase
Copy link

This wasn't added in? I had to manually make the changes.. would be great to have this in master.

@tyzoid
Copy link
Collaborator

tyzoid commented Jul 18, 2017

The main issue is that this is how the output format is defined, for ex:

/api/endpoint.json => returns a json object
/api/endpoint.xml  => returns an xml object

This could be solved by changing the way that the format string is handled, but requires careful attention. As is, this PR would essentially break this functionality.

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.

None yet

3 participants