FirebaseExtended / FirebaseExtended/reactfire

useFirestoreCollection triggers a double render

Ouverte
#484 4 commentaires 2 réactions 0 personnes assignées Voir sur GitHub
v5
Langage dominant
TypeScript
Étoiles
3.6k
Forks
403
Merge moyen
14 h 53 min
PR mergées (30 j)
5

Description

Hi,

Seems like `useFirestoreCollection` always triggers unnecessary render when fetching data (suspense is **on**). Similar behavior is also the case for `useUser`.
There's a chance I'm simply using the API wrong, so please advise if so :)

### Version info

**React:** 17.0.2

**Firebase:** 9.1.3

**ReactFire:** 4.2.0

### Test case

```javascript
const q = /* whatever query */;
const queryResults = useFirestoreCollection(q);
console.log(queryResults);

```

### Expected behavior

`console.log` is called once.

### Actual behavior

`console.log` is called twice.

Ultimately, what I'm trying to achieve is a code similar to the one below

```javascript
const queryResults = useFirestoreCollection(query);
return queryResults.data.docs.map((d) => /* render */);
```

that renders only once when the data is fetched (and then only when there're changes in the collection).

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par reproduire le comportement avec le cas de test useFirestoreCollection et comparez-le au comportement similaire de useUser. Inspectez les points d’entrée de ces hooks afin de déterminer pourquoi la récupération avec Suspense provoque l’exécution de console.log deux fois. Le travail est terminé lorsque la récupération des données produit un rendu, tandis que les rendus suivants ne se produisent que lorsque la collection change.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, react
Domaine
database, frontend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.