-
Notifications
You must be signed in to change notification settings - Fork 19
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
ISSUE-155: Adds JSON Patch Action example #162
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reads really well @DiegoPino, is a great explanation and walkthrough for JSON Patch. I made a few small potential formatting suggestions, but may/may not all be applicable.
Co-authored-by: Allison Lund <[email protected]>
Co-authored-by: Allison Lund <[email protected]>
Co-authored-by: Allison Lund <[email protected]>
Co-authored-by: Allison Lund <[email protected]>
Co-authored-by: Allison Lund <[email protected]>
Co-authored-by: Allison Lund <[email protected]>
Thanks @alliomeria gosh so many issues. Thanks for your sharp eyes on this |
Co-authored-by: Albert Min <[email protected]>
Co-authored-by: Albert Min <[email protected]>
Co-authored-by: Albert Min <[email protected]>
Co-authored-by: Albert Min <[email protected]>
@aksm thanks for your corrections. Ok will merge. Let's see how this looks on MKDocs (if it looks at all or not!) |
See #155
@aksm @alliomeria any of you could give this a quick typo-proof read and suggest directly code/changes as comments?
Gracias