Change route fallback action to PATCH instead of POST #223

Closed
asm89 opened this Issue Apr 4, 2012 · 3 comments

Projects

None yet

3 participants

@asm89
Member
asm89 commented Apr 4, 2012

The route loader will generate a route with the action 'POST' for each action that is not a predefined one. The docs state the following about this:

All actions that do not match the ones listed in the sections above will register as a POST action.
...
This allows for easy updating of aspects of a resource, without having to deal with the resource as a whole at the standard POST endpoint.

Following the reasoning of allowing for easier updating aspects of a resource, the fallback action should rather be PATCH instead?

ping @everzet If you agree I'll send a PR updating the route loader, tests and documentation.

@everzet everzet was assigned Apr 4, 2012
@Dattaya
Contributor
Dattaya commented Apr 4, 2012

And an interesting article about PATCH - Edge Rails: PATCH is the new primary HTTP method for updates - http://weblog.rubyonrails.org/2012/2/25/edge-rails-patch-is-the-new-primary-http-method-for-updates/

@everzet
Member
everzet commented Apr 4, 2012

ok

@asm89
Member
asm89 commented Apr 4, 2012

👍 I'll send a PR.

@asm89 asm89 closed this in b48f159 Apr 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment