search

instantdb / instant

The realtime client-side database
https://instantdb.com
Apache License 2.0
5.85k stars 137 forks source link

[core][merge] Fix client deep-merge null logic #205

Closed markyfyi closed 4 days ago

markyfyi commented 5 days ago

Fixes https://github.com/instantdb/instant/issues/54. I added a util to the patch example to replicate the case outlined.

Our logic in immutableDeepMerge was plain wrong, and we weren't deleted properties that were "nulled" out.

github-actions[bot] commented 5 days ago

View Vercel preview at instant-www-js-core-fix-deep-merge-null-jsv.vercel.app.