-
Notifications
You must be signed in to change notification settings - Fork 186
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 "create" operation #10
base: master
Are you sure you want to change the base?
Conversation
I agree that this operation is nice, but indeed I'd like to keep RFC 6902 compatibility, which means I'll need to refactor and make As to the patch itself, it looks good, but I'd suggest the following changes:
|
Thanks! I'll make an update using your suggestions. |
Refactor AdditionOperation
Squashed |
Updated to make |
Excellent! Now I need to find the time to work on that project again... Right now I'm busy with json-schema-*... |
Thanks, I'll keep an eye out for any updates. |
…eration Add remove? operation
…son_pointer_double_slashes #807 JsonPointer double slashes validation
I'd like to extend JSON Patch and add a new "create" operation. It is exactly like "add", except it will not replace an existing value at the requested path. It will be useful in situations in where a client wants to ensure that they are not overwriting a value set by another client.
This was a suggestion from an email thread with the RFC6902 authors:
Although it deviates from the RFC, I think it would be useful for others to use, perhaps in a custom JsonPatch implementation as you mentioned in #9.
Let me know what you think!