Open bruceharrison1984 opened 1 year ago
I tried to use the library for the first time today and hit upon this issue. I think the cause is due to the call to queryEqual here: https://github.com/FirebaseExtended/reactfire/blob/3cb446838d68e7ab9264c549fe7f1dccaf166881/src/firestore.tsx#L16
In my case, I am minting new converter objects on each render, which breaks the equality check, with something like:
Firestore.collection(firestore, prefix, 'doc').withConverter(
docConverter<Doc>()
),
If I instead save off the converter to a top-level constant, which means the equality check can presumably rely on identity for the converter part, I can see the collection promise gets fulfilled and my app renders correctly.
It's something like this saved off outside of the component lifecycle, in my case at the top level of a module:
const docConverter = docConverter<Doc>()
Then during a render, it's reusing the object:
Firestore.collection(firestore, prefix, 'doc').withConverter(
docConverter
),
I think this is not a reactfire bug, but I think a common use-case that should be documented as I wasted hours on it and I imagine others would, too.
Another approach to consider is to allow users to pass in a query key override, like react-query (the lib I was coming from), if it's too difficult to track whether equality will work out from reading subtle code changes.
Version info
React: 18.2.0
Firebase: 9.13.0
ReactFire: 4.2.2
Test case
Steps to reproduce
Using
useFirestoreCollectionData
with a query that has a convertor attached results in an infinite loop of undefined. Removing thewithConvertor
call from the query immediately fixes the issue. The bug seems to occur whether attaching the convertor to the query or the collection.Expected behavior
I would expect the query to succeed regardless of if a convertor is utilized or not. Currently, I can only return an untyped collection and then have to map it in an additional step. You can also force the datatype by using an
as
clause at the point of usage:but this also feels unnecessary.
Single documents work fine with convertors.
Actual behavior
The query loops forever, returning zero results.