docs(Firestore): Adding examples for v22#8971
docs(Firestore): Adding examples for v22#8971Bilal-Abdeen wants to merge 7 commits intoinvertase:mainfrom
Conversation
|
@Bilal-Abdeen is attempting to deploy a commit to the Invertase Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
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.
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>
Bilal-Abdeen
left a comment
There was a problem hiding this comment.
Removed old examples (before v22)
Description
Related issues
Release Summary
Checklist
AndroidiOSOther(macOS, web)e2etests added or updated inpackages/\*\*/e2ejesttests added or updated inpackages/\*\*/__tests__Test Plan
Think
react-native-firebaseis great? Please consider supporting the project with any of the below:React Native FirebaseandInvertaseon Twitter