JsonPatch icon indicating copy to clipboard operation
JsonPatch copied to clipboard

Look into non-generic overloads for creating ops & non-generic JsonPatchDoc

Open KevinDockx opened this issue 8 years ago • 3 comments

Might conflict with JsonPatch.Dynamic?

cfr https://github.com/KevinDockx/JsonPatch.Dynamic/issues/10#issuecomment-273648173

KevinDockx avatar Jan 19 '17 16:01 KevinDockx

The two non-generic JsonPatchDocument classes would be in different namespaces, so the "conflict" is manageable.

HappyNomad avatar Jan 27 '17 21:01 HappyNomad

Yep, but the thing is that we'd end up with two classes that are copies of each other but just live in a different namespace - and that's not right. The best way to solve this from a design POV would be to move the non-generic JsonPatchDoc to this lib, and remove it from JsonPatch.Dynamic altogether, but that means a breaking change in JsonPatch.Dynamic. That could be mitigated by marking it as obsolete in JsonPatch.Dynamic (and remove it altogether later on) - what do you think?

KevinDockx avatar Jan 28 '17 09:01 KevinDockx

That approach sounds fine to me.

HappyNomad avatar Jan 28 '17 16:01 HappyNomad