fix(ssr): use ssrKey in firestore

vuejs · Nov 22, 2022 · 25d86ca · 25d86ca
1 parent c4c815c
commit 25d86ca
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/src/firestore/index.ts b/src/firestore/index.ts
@@ -132,7 +132,11 @@ export function _useFirestoreRef(
   // only add the first promise to the pending ones
   // TODO: can we make this tree shakeable?
   if (initialSourceValue) {
-    removePendingPromise = addPendingPromise(promise.value, initialSourceValue)
+    removePendingPromise = addPendingPromise(
+      promise.value,
+      initialSourceValue,
+      options.ssrKey
+    )
   }
 
   // TODO: SSR serialize the values for Nuxt to expose them later and use them

diff --git a/src/ssr/plugin.ts b/src/ssr/plugin.ts
@@ -42,7 +42,7 @@ export function addPendingPromise(
   } else {
     // TODO: warn if in SSR context in other contexts than vite
     if (process.env.NODE_ENV !== 'production' /* && import.meta.env?.SSR */) {
-      console.warn('[VueFire]: Could not get the path of the data source')
+      console.warn('[VueFire SSR]: Could not get the path of the data source')
     }
   }
 

diff --git a/tests/firestore/collection.spec.ts b/tests/firestore/collection.spec.ts
@@ -307,10 +307,12 @@ describe(
     }
 
     it('can be bound to a ref of a query', async () => {
-      const { showFinished, listToDisplay } = await createFilteredLists()
+      const { showFinished, listToDisplay, listRef } =
+        await createFilteredLists()
 
       const { wrapper, data, promise } = factoryQuery({
         ref: listToDisplay,
+        options: { ssrKey: 'list' },
       })
 
       await promise.value