Compatibility with external deep proxies (rebased)

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

Compatibility with external deep proxies (rebased) #43

Closed warpech closed 4 years ago

warpech commented 5 years ago

This is PR #39 rebased on current master.

warpech commented 5 years ago

Reason this was not merged:

I am not merging this until we find it absolutely necessary for view binding.

One thing that I find very annoyning about this change is how it affects debugging. Any attempt to read a property from the observed object causes the debugger to step into the trapForGet.

Copied from https://github.com/Palindrom/JSONPatcherProxy/pull/39#issuecomment-500766287

warpech commented 4 years ago

Closing because this PR turned out to be not needed