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

Patch method support #121

Open
Satak opened this issue Feb 23, 2016 · 5 comments
Open

Patch method support #121

Satak opened this issue Feb 23, 2016 · 5 comments

Comments

@Satak
Copy link

Satak commented Feb 23, 2016

Does node-restful support 'Patch' REST method?

@manicExpressive
Copy link

Sadly, it looks like that's not yet implemented:

var methods = ['get', 'post', 'put', 'delete'], // All HTTP methods, PATCH not currently supported

@baugarten
Copy link
Owner

This is true

@camilomontoyau
Copy link

But if you only send the property you want to change via put, it only chages it and persist the other ones

@OmgImAlexis
Copy link

Currently PATCH is supported but it's called PUT, PUT should actually override the whole object with the new one that's provided whereas PATCH should only change the field you provide. It looks like @baugarten got these backwards.

@OmgImAlexis
Copy link

If you guys want to try this out I've opened a PR to possibly fix this.
Ref: #152

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

No branches or pull requests

5 participants