Skip to content
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

[breaking change] Do not replace array.length as patch when removing items #964

Closed
wants to merge 1 commit into from

Conversation

kshramt
Copy link

@kshramt kshramt commented Jul 31, 2022

Patches generated by the current implementation do not follow the RFC method for removing an element from an array (https://www.rfc-editor.org/rfc/rfc6902#page-13) and cannot be used from other languages (e.g., https://pypi.org/project/jsonpatch/).

@netlify
Copy link

netlify bot commented Jul 31, 2022

Deploy Preview for quizzical-lovelace-dcbd6a canceled.

Name Link
🔨 Latest commit 503b4be
🔍 Latest deploy log https://app.netlify.com/sites/quizzical-lovelace-dcbd6a/deploys/62e694b0eace7d0009cb9481

Patches generated by the current implementation do not follow the RFC method for removing an element from an array (https://www.rfc-editor.org/rfc/rfc6902#page-13) and cannot be used from other languages (e.g., https://pypi.org/project/jsonpatch/).
@kshramt
Copy link
Author

kshramt commented Jul 31, 2022

#208 (comment) might be resolved by the PR.

@mweststrate
Copy link
Collaborator

Thanks for this PR! Apologies for the slow follow up, I do intend to merge, but as a separate major version.

@mweststrate mweststrate changed the title Do not replace array.length [breaking change] Do not replace array.length as patch when removing items Jan 15, 2023
mweststrate added a commit that referenced this pull request Mar 23, 2023
@mweststrate
Copy link
Collaborator

Branch has been merged manually into #1028, thanks!

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.

None yet

2 participants