feat: Check if client exists in persist

rocicorp / replicache

Realtime Sync for Any Backend Stack

https://doc.replicache.dev

1.05k stars 37 forks source link

feat: Check if client exists in persist #867

Closed arv closed 2 years ago

arv commented 2 years ago

We now check if the client ID exists in the client map when we do a persist. If it doesn't we throw a MissingClientError.

For testing purpuse we can skip this check.

The intended use is to handle clients that are missing and raise an "event" on the Replicache instance when this happens.

Towards #784

vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/rocicorp/replicache/B8ovfx2b1JwaQxr7gxQhtCfBHw4J
✅ Preview: https://replicache-git-arv-check-for-missing-client-rocicorp.vercel.app

grgbkr commented 2 years ago

LGTM