use store.snapshot for getServerSnapshot

tim-smart · Dec 19, 2023 · f005658 · f005658
1 parent 6fbff2c
commit f005658
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.changeset/eleven-seals-hammer.md b/.changeset/eleven-seals-hammer.md
@@ -0,0 +1,5 @@
+---
+"@effect-rx/rx-react": patch
+---
+
+use store.snapshot for getServerSnapshot
diff --git a/packages/rx-react/src/index.ts b/packages/rx-react/src/index.ts
@@ -40,7 +40,6 @@ export const RegistryContext = React.createContext<Registry.Registry>(Registry.m
 interface RxStore<A> {
   readonly subscribe: (f: () => void) => () => void
   readonly snapshot: () => A
-  readonly serverSnapshot: () => A
 }
 
 const storeRegistry = globalValue(
@@ -60,9 +59,6 @@ function makeStore<A>(registry: Registry.Registry, rx: Rx.Rx<A>): RxStore<A> {
     },
     snapshot() {
       return registry.get(rx)
-    },
-    serverSnapshot() {
-      return undefined as any
     }
   }
   stores.set(rx, newStore)
@@ -71,7 +67,7 @@ function makeStore<A>(registry: Registry.Registry, rx: Rx.Rx<A>): RxStore<A> {
 
 function useStore<A>(registry: Registry.Registry, rx: Rx.Rx<A>): A {
   const store = makeStore(registry, rx)
-  return React.useSyncExternalStore(store.subscribe, store.snapshot, store.serverSnapshot)
+  return React.useSyncExternalStore(store.subscribe, store.snapshot, store.snapshot)
 }
 
 /**