Skip to content

docs(Firestore): Adding examples for v22#8971

Open
Bilal-Abdeen wants to merge 7 commits intoinvertase:mainfrom
Bilal-Abdeen:patch-13
Open

docs(Firestore): Adding examples for v22#8971
Bilal-Abdeen wants to merge 7 commits intoinvertase:mainfrom
Bilal-Abdeen:patch-13

Conversation

@Bilal-Abdeen
Copy link
Copy Markdown
Contributor

Description

Related issues

Release Summary

Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
    • [x ] Yes
  • My change supports the following platforms;
    • [x ] Android
    • [x ] iOS
    • [x ] Other (macOS, web)
  • My change includes tests;
    • e2e tests added or updated in packages/\*\*/e2e
    • jest tests added or updated in packages/\*\*/__tests__
  • I have updated TypeScript types that are affected by my change.
  • This is a breaking change;
    • Yes
    • [x ] No

Test Plan


Think react-native-firebase is great? Please consider supporting the project with any of the below:

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 13, 2026

@Bilal-Abdeen is attempting to deploy a commit to the Invertase Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Firestore usage documentation to include code examples for the modular API introduced in React-Native-Firebase v22, providing side-by-side comparisons with the legacy namespaced API. The review feedback highlights several issues in the new code snippets, specifically missing database initialization (db = getFirestore()) and missing imports in certain examples. Additionally, it is noted that initializeFirestore should be awaited to ensure configuration settings are properly applied before further interactions.

Comment thread docs/firestore/usage/index.md Outdated
Comment thread docs/firestore/usage/index.md Outdated
Comment thread docs/firestore/usage/index.md Outdated
Comment thread docs/firestore/usage/index.md
Bilal-Abdeen and others added 6 commits April 13, 2026 11:29
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@Bilal-Abdeen Bilal-Abdeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed old examples (before v22)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant