Closed kurtcarpenter closed 3 years ago
Would appreciate feedback on the _isDefaultSetToUndefined
checks as I'm less confident in those changes.
Nice stuff, will review later today! Thanks for bearing with some of the weird recursion, I'm looking to simplify it greatly it in the next few weeks.
This looks good. I think some of the _isDefaultSetToUndefined logic is overkill (_isDefaultSetToUndefined currently only gets set on subdocument array schemas) but it isnt harming anything so I wouldnt worry about it right now. This file needs a big cleanup as a whole so no use in trying to refactor too much now.
Add support for many other ways of defining Object and Array types, documented here: Array and Object
parseKey
intoparseKeyFn
for reusable invocation and ability to unit test_isDefaultSetToUndefined