ajv-merge-patch icon indicating copy to clipboard operation
ajv-merge-patch copied to clipboard

Published 20 hours ago • verified publisher icon ajv-validator

feat: support for arrays of mergeable objects in the “with” argument of “$merge”

Open azaslavsky opened this issue 8 years ago • 4 comments

This is a working implementation of the feature proposed in #14. It would also close #12.

azaslavsky avatar Oct 10 '17 21:10 azaslavsky

Coverage Status

Coverage remained the same at 100.0% when pulling e84920a2b42c59d29553ffa5b05fc9c0b0f01954 on azaslavsky:arrays-for-with into d847ec318fad1ff7bad2a183e3329923fad8a70e on epoberezkin:master.

coveralls avatar Oct 10 '17 21:10 coveralls

@azaslavsky Thank you, I will review

epoberezkin avatar Oct 11 '17 14:10 epoberezkin

Hello, I was looking for the fix to #12 and was wondering, is there an update on the PR review?

vganapat avatar Nov 14 '17 18:11 vganapat

The fix to #12 is very simple, there is a separate PR for that. To support an array instead of a simple schema, there is no need to change the package completely. Instead of passing "json-merge-patch" package as is, you just need to pass a wrapper around this package that understands arrays.

epoberezkin avatar Mar 11 '18 09:03 epoberezkin