Skip to content

Conversation

pinkeshmars
Copy link
Collaborator

Description

Update Firebase integration setup + Add FAQ

Linear ticket and magic word Fixes DEVR-1129

Type of change

  • Typo fix
  • New feature
  • Enhancement to current docs
  • Removed outdated references
  • Update assets

@leighajarett
Copy link
Contributor

I think there are some FAQs we might want to add:

  1. If you see a runtime error like "[cloud_firestore/failed-precondition] The query requires an index" ... I think the fix is that you should click the corresponding link in the logs to generate the index and then wait for it to be complete. Please try and test this out to show a specific example / make sure these instructions are correct

  2. If you see a runtime error like "[cloud_firestore/permission-denied] Missing or insufficient permissions" then you either need to redeploy the rules (in the firebase panel) or update the rules to be correct (manually or with agent) and then redeploy

@pinkeshmars
Copy link
Collaborator Author

@leighajarett the FAQs you suggested are added now! You can take a look again.

</div>
<p></p>

### First-Time Firebase Project Setup
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
### First-Time Firebase Project Setup
### Initial Firebase Project Setup


:::info

This process only inserts sample data into the **Firestore Database** for the specified user email. It does **not** create a user in Firebase Authentication. Make sure the user with the specified email exists in Firebase Authentication to be able to access the data.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can be more specific about how to add a user to firebase authentication?

</div>
<p></p>

## Add Sample Data to Firebase
Copy link
Contributor

Choose a reason for hiding this comment

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

Mention that you can only do this once?


![add-sample-data.avif](imgs/add-sample-data.avif)

## Enable Billing
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you put the error message that users would see somewhere? Users might go to the agent or google search with that exact error message so want to tie it to this section somehow



<details>
<summary> Why do cloud function deployments fail? </summary>
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment as before about adding examples of specific errors


If the generated client code includes authentication-related functionality, you must configure sign-in providers in the Firebase Console to make it work.

To enable authentication, click **Configure Authentication**. This opens the Authentication page in the Firebase Console, where you can select a sign-in provider such as Email/Password, Google, or Facebook. Enable the chosen provider, configure its settings (for example, OAuth credentials for Google or Facebook), and click **Save**. Repeat this process to enable any additional providers required by your app.
Copy link
Contributor

Choose a reason for hiding this comment

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

I submitted another devrel ticket in linear about oauth flows, previously the URL for the preview would change but now it should be stable to - please test and confirm and add some info about getting Google / Facebook auth to work with firebase in your app... I think we need to talk about whitelisting the preview URLs for testing purposes

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure! I will look into it. And it looks like the review comments weren’t submitted earlier. No worries — I’ll take care of them in a new PR.

@leighajarett leighajarett merged commit b47adf3 into main Oct 3, 2025
1 check passed
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.

3 participants