[BUG] Duplicate entries when lazyMode:true

[ucefkh]

This is a Serious bug I saw for a few years now that wasn't fixed yet

Reproduction:
Use react admin-firebase and set lazyMode:true that's it!

I can't share my Firebase app publicly so you can try it

Look how bad this is!

It is affecting us so much and I knew this bug for a few years now that was never fixed

[ucefkh]

[ucefkh]

seems people know about it and made this

https://www.npmjs.com/package/react-admin-firebase-lazy-load/v/3.2.32

[ucefkh]

Hello guys,

Any updates on this please?

[amilosmanli]

@ucefkh I'm using the lazy loading as well, but never encountered this issue before.

Can you share your options that you pass to the FirebaseDataProvider ?

lazyMode:true is not the correct option btw, you should use

lazyLoading: {
    enabled: true,
  },

[quynguyencanalcircle]

Hi @ucefkh, I got the same problem.
I make a patch to fix on my side:

ResourceManager.ts:95

newDocs.forEach((d) =>{
      const data = parseFireStoreDocument<IResourceItem>(d);
      const existId = resource.list.findIndex(i => i.id === data.id);
      if(existId>=0) {
        resource.list[existId] = data
      } else {
        resource.list.push(data)
      }
    });

Hope this help!

[KonkretneKosteczki]

I get the same error even without lazy loading enabled. With the newest version of the react admin (4.9.3) and react admin firebase (4.1.2) Every time i refresh the page, entries get appended to the old ones.

[ucefkh]

yes @KonkretneKosteczki I confirm this is still a bug in 4.1.2 latest version!! so weird!

even though they got getServerCounter now!

[ucefkh]

Seems the issue comes from the getQueryCursor which doesnt find the last document id cursor so it fetches the docs again and again

but also we might need to enable indexing for fields??

https://firebase.google.com/docs/firestore/query-data/aggregation-queries

[ucefkh]

[ucefkh]

[ucefkh]

I fixed this issue on my fork

https://www.npmjs.com/package/react-admin-firebase-01tek

the solution was so dumb!