-
Notifications
You must be signed in to change notification settings - Fork 10
Add Firebase integration docs #5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces Firebase integration documentation and restructures the documentation organization with a new categorical sidebar structure.
Key changes include:
- Addition of comprehensive Firebase integration documentation with setup, configuration, and deployment guidance
- Implementation of a structured sidebar with Test & Publish and Integrations categories
- Addition of mobile testing documentation and Google Play Store deployment guides
- Cleanup of legacy redirect configurations in firebase.json
Reviewed Changes
Copilot reviewed 10 out of 18 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| sidebars.ts | Restructures sidebar from autogenerated to categorical organization with Test & Publish and Integrations sections |
| firebase.json | Removes extensive legacy redirect rules, keeping only blog redirect |
| docs/integrations/firebase.md | Adds comprehensive Firebase integration guide covering connection, setup, code generation, and deployment |
| docs/test-and-publish/test/test-on-mobile-device.md | Provides detailed mobile device testing instructions for Android and iOS |
| docs/test-and-publish/publish/pre-checks-publishing.md | Documents essential pre-publishing checklist and quality assurance steps |
| docs/test-and-publish/publish/google-playstore-deployment.md | Complete Google Play Store deployment guide with service account setup |
| docs/index.md | Minor update removing broken link reference |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…firebase-integration
PoojaB26
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested some changes, please look into it asap.
docs/integrations/firebase.md
Outdated
|
|
||
|  | ||
|
|
||
| ### What happens during code generation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldnt have a question as a header
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pinkeshmars i didn't mean add a question mark. i mean headers shouldn't be a "question".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops! Sorry, I misread it. fixed now.
…tterFlow/dreamflow-documentation into feature/firebase-integration
|
@PoojaB26 review comments are addressed! You can take a look again. |
…entation into feature/firebase-integration
…entation into feature/firebase-integration
…tterFlow/dreamflow-documentation into feature/firebase-integration
Description
Add Firebase integration docs
Linear ticket and magic word Fixes DEVR-1091
Type of change