bypass jsonpatch

[rmorshea]

The jsonpatch library is broken and it seems like there's no one to maintain it. We should just avoid JSON Patch entirely since, so long as you're careful about what components you re-render, diffing is not usually necessary. In a future release, the client will stop using JSON patch altogether.

Checklist

Please update this checklist as you complete each item:

• Tests have been included for all bug fixes or added functionality.
• The changelog.rst has been updated with any significant changes.
• GitHub Issues which may be closed by this Pull Request have been linked.

[Archmonger]

How certain are we that the current exception (onChange missing) detected on reactive-python/reactpy-django#86 is related to Json patch?

[rmorshea]

I'm pretty certain. Could you test using this branch though to be sure? pip install git+https://github.com/idom-team/idom@bypass-jsonpatch

[Archmonger]

This does not fix the issue on django-idom, tested via pip install git+

[rmorshea]

I have a couple questions:

• on what branch is this happening?
• with what code sample?
• is the error message the same?

[Archmonger]

Branch

reactive-python/reactpy-django#86 (idom-team/django-idom@use_database)

Code Sample

Manually running the use_mutation and use_database tests via python test/manage.py runserver and inspecting in browser

Error message

Yes it is the same even when using this bypass jsonpatch branch.

[Archmonger]

Confirmed working.

[rmorshea]

will release tomorrow