Allow `useFirestoreDoc` `DocumentReference` arguments to be nullable

Offen

#463 erstellt am 23.9.2021 13 Kommentare 18 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 52/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Ruhig
Tech-Stack: react, typescript
Bereich: databases, frontend

Rechercherichtung

Beginne am useFirestoreDoc-Einstiegspunkt und vergleiche das Verhalten seiner Argumente mit dem in der Issue beschriebenen nullable-Verhalten von useDocument. Verfolge, wie der TypeScript-Typ DocumentReference behandelt wird, und prüfe alle vorhandenen Firestore-Hook-Tests. Als erledigt gilt die Aufgabe, wenn nullable-Referenzen ohne einen Invalid-Path-Fehler akzeptiert werden und der Hook weiterhin gültig ist, wenn er bedingungslos aufgerufen wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

react-firebase-hooks allows the document reference to useDocument to be nullable and just returns undefined if it is. This was useful for paths relying on nullable info, like auth.currentUser.uid:

const [snapshot, loading, error] = useDocument(auth.currentUser && firestore.doc(`users/${auth.currentUser.uid}`));

where if the user were not signed in there wouldn't be an error thrown since useDocument would just return undefined. It would be nice if a similar feature were added to reactfire, as

const { status, data: firebaseDoc } = useFirestoreDoc(auth.currentUser && doc(firestore, 'users', auth.currentUser.uid));

won't work as the type of ref is DocumentReference, not DocumentReference | null,

const { status, data: firebaseDoc } = useFirestoreDoc(doc(firestore, 'users', auth.currentUser?.uid));

will throw an error when the user is not signed in (as the path will become invalid), and

if (auth.currentUser) {
    const { status, data: firebaseDoc } = useFirestoreDoc(doc(firestore, 'users', auth.currentUser.uid));
}

violates the rules of hooks.
Currently relying on a rather abhorrent workaround to resolve this and it would be ideal if reactfire could support this behavior natively.

Vorherrschende Sprache: TypeScript
Sterne: 3.6k
Forks: 403
Ø Merge: 5 T. 1 Std.
Gemergte PRs (30 T.): 10

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus FirebaseExtended/reactfire

No bindings for Firebase AI Logic Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 38/100

FirebaseExtended/reactfire#801 · Erstellt am 25.8.2026
Suspense mode resumes with the previous observable's value on React 19 Offen

v5

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

FirebaseExtended/reactfire#793 · Erstellt am 12.8.2026
The observable cache leaks subscriptions and is shared across SSR requests Offen

v5

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

FirebaseExtended/reactfire#790 · 2 Kommentare · Erstellt am 11.8.2026
React 19 foundation: run tests against 19, then drop React 16/17 Offen

v5

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100

FirebaseExtended/reactfire#789 · Erstellt am 11.8.2026
Tests that keep passing when the code they cover is broken Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100

FirebaseExtended/reactfire#788 · Erstellt am 10.8.2026

Alle Issues in FirebaseExtended/reactfire