Skip to content

Conversation

@mvcatsifma
Copy link
Collaborator

Support JSON Merge Patch

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If jshint is quick we could add this to the default task.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@jcassee
Copy link
Owner

jcassee commented Oct 12, 2015

One more thing that is missing is an update of the README to document the $patch method. I can also write that if you want.

Thomas Delnoij added 3 commits October 13, 2015 10:43
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point using request.data here and not data from the arguments, by the way!

jcassee added a commit that referenced this pull request Oct 13, 2015
@jcassee jcassee merged commit 00988f0 into master Oct 13, 2015
@jcassee jcassee deleted the feature/support-json-patch branch October 13, 2015 12:44
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.

3 participants