Skip to content

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.

2 participants