egeriis / laravel-jsonapi

Make it a breeze to create a jsonapi.org compliant APIs with Laravel 5.
MIT License
146 stars 27 forks source link

Use PATCH instead of PUT #23

Closed mfn closed 9 years ago

mfn commented 9 years ago

As per http://jsonapi.org/format/1.0/#crud-updating , PATCH is the actual verb to use, not PUT.

See also the FAQ: http://jsonapi.org/faq/#wheres-put

egeriis commented 9 years ago

Haven't seen that update. I am quite sure it's a simple fix to implement this. Feel free to open a PR!

egeriis commented 9 years ago

Ok if I close this? Or do you want to add something?

mfn commented 9 years ago

Yes, thanks, it's OK to close.

Please note however that I currently do not plan to implement handlePatchDefault(), thus the references PR is not a complete PATCH implementation, merely a start for whoever wants to complete it.

egeriis commented 9 years ago

Yup, that's perfect.