You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I removed some usages of JSON.NET down to where our only usage is pretty much the JSON patch support.
Spent some time taking a look at json-everything's JsonPatch support and it only uses JSON Pointers for paths and does not support some of the things we are doing where we are allowing full JSON Path support where we can do things like $.books[?(@.author == 'John Steinbeck')] to remove all array elements from the books array that has an author sub-property that matches John Steinbeck. So basically we would need to make major changes to their libraries or talk them into supporting JSON Path expressions even though the unofficial JSON Patch spec only supports JSON Pointer.
For Json Patching it would be nice to see if we can get it to work without a dependency or we can internalize the dependency. I see several spots doing patching
I removed some usages of JSON.NET down to where our only usage is pretty much the JSON patch support.
Spent some time taking a look at json-everything's
JsonPatch
support and it only uses JSON Pointers for paths and does not support some of the things we are doing where we are allowing full JSON Path support where we can do things like$.books[?(@.author == 'John Steinbeck')]
to remove all array elements from the books array that has an author sub-property that matchesJohn Steinbeck
. So basically we would need to make major changes to their libraries or talk them into supporting JSON Path expressions even though the unofficial JSON Patch spec only supports JSON Pointer.Originally posted by @ejsmith in #96 (comment)
Found this issue where someone asked him and he said no to supporting JSON Path.
Originally posted by @ejsmith in #96 (comment)
Comment for previous history:
For Json Patching it would be nice to see if we can get it to work without a dependency or we can internalize the dependency. I see several spots doing patching
Originally posted by @niemyjski in #96 (comment)
The text was updated successfully, but these errors were encountered: