Palindrom / JSONPatcherProxy

ES6 proxy powered JSON Object observer that emits JSON patches when changes occur to your object tree.
94 stars 14 forks source link

Imported `Operation` type from fast-json-patch #32

Closed jonahbron closed 5 years ago

jonahbron commented 5 years ago

This changes the return type of generate() to use the Operation type from fast-json-patch, since this project depends on that one. It also uses that Operation type for the observe callback.

alshakero commented 5 years ago

It's getting hard to keep track with this number of PRs. Can you please merge them all in 1 PR?

jonahbron commented 5 years ago

@alshakero I've started by splitting each change into a separate PR so you can select what you think is appropriate to merge, as these are all largely unrelated changes. It might be easiest for you to simply merge the ones you like, and close the rest.

However if you wish to wholesale accept any changes I propose, I'm happy to merge these all together for you. lmk 😃

jonahbron commented 5 years ago

@alshakero Checking in on this. Have you decided if you want them merged all together, or do you want to maintain the ability to pick-and-choose?

alshakero commented 5 years ago

Thank you so much for all the PRs!