-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add traverse operation #21
Comments
I prefer this over #23 because it does not modify JSON Pointers spec. But it also looks very similar to #9. And similarly, I am against it because:
So this proposal just changes that you do not have to write And similarly to this comment, I think:
I think JSON patch should be for a fixed base JSON to apply to, and if the base JSON changes, another algorithm should determine how to try to fuzzy-apply the JSON patch, instead of making the JSON patch itself more complicated. |
I see it as a response on this old blog post: http://susanpotter.net/blogs/software/2011/07/why-json-pointer-falls-short/ (which is about JSON Pointer and querying, but for our use-case Patch and modifying is more important, nevertheless I'd be happy to hear about state of art for querying)
Basically, given this input array:
and following imaginary operation:
would produce:
traverse
here should roughly correspond tomap
(I like this name even more) from FP-languages.The text was updated successfully, but these errors were encountered: