Repository navigation
Expand Firebase page with setup, Auth, and database guides - #13980
Conversation
…rt backend guides
|
Staged preview of the updated docs.flutter.dev site (updated for commit a100662): |
- Remove Go from the Cloud Functions for Firebase language list; Firebase ships SDKs for Node.js (JavaScript/TypeScript), Python, and Dart. - Note that Dart support for Cloud Functions is experimental and requires enabling the dartfunctions experiment in the Firebase CLI. - Generalize the platform list in the intro so it covers Windows (beta). - Clarify that SignInScreen in the auth example is a placeholder, since FirebaseUI exports a widget with the same name. - Replace 'vs.' per the style guide.
The detailed SQL Connect pitch already lives on the Choose a database page; keep this page to a short summary plus links.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the Firebase documentation and the LLM resource list to provide a comprehensive guide on connecting Flutter apps to Firebase, covering authentication, Firestore, SQL Connect, and serverless Dart functions. The review feedback correctly identifies several naming inaccuracies that need to be addressed, specifically updating 'Firebase SQL Connect' to 'Firebase Data Connect' and 'Firebase AI Logic' to 'Vertex AI in Firebase' (along with its package firebase_vertexai). Additionally, the Firestore code example should be refactored from a StatelessWidget to a StatefulWidget to avoid the performance anti-pattern of recreating the stream subscription inside the build method.
|
@jesskuras - could you check the Gemini comments, I can review this afterwards! |
Create the query stream once in State rather than on every rebuild, so StreamBuilder doesn't resubscribe each time the widget rebuilds.
Done! There was only one legitimate one, but I made that change real quick. :) |
|
@jesskuras - left my feedback! The only blocking piece is the offline first part on firestore. Otherwise, LGTM! |
…sentence - Describe Cloud Firestore as offering offline caching and persistence rather than being offline-first. - Drop 'from your schema and queries' from the SQL Connect summary.
Replace 'offline-first' with 'offline caching' to match the Firebase page and Cloud Firestore's actual capabilities.
Amazing, thanks a bunch! |
Description of what this PR is changing or adding, and why:
Expands
/data-and-backend/firebasewith on-domain setup instructions, code examples, and architectural guidance for building Flutter apps with Firebase:cloud_firestore,firebase_data_connect,firebase_auth,firebase_storage,firebase_ai,cloud_functions/firebase_functions, andfirebase_crashlytics).flutterfire_cli,flutterfire configure, andFirebase.initializeApp().StreamBuilderauth-state example and email/password sign-in.Firebase for Flutterdescription inllms.txt.Issues fixed by this PR (if any):
PRs or commits this PR depends on (if any):
Presubmit checklist