Skip to content

Update firebase-react-native.md #6261

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

Merged
merged 3 commits into from
Jun 30, 2025
Merged

Conversation

PcooperSegment
Copy link
Contributor

Proposed changes

Changing the link, The old link refers back to our old React Native Library, which is no longer supported. I've changed some wording around as well to make sure customers know that installing the Google services is needed.

Merge timing

  • ASAP once approved

Changing the link, The old link refers back to our old React Native Library which is no longer supported. I've changed some wording around as well to make sure customers know that installing the google services is needed.
Changed to recommend links from Niall
@PcooperSegment
Copy link
Contributor Author

Hi @niallzato Just changed to recommended links.

@PcooperSegment PcooperSegment marked this pull request as ready for review January 31, 2025 08:49
@PcooperSegment PcooperSegment requested a review from a team as a code owner January 31, 2025 08:49
…destination-plugins/firebase-react-native.md
@forstisabella forstisabella enabled auto-merge March 21, 2025 15:50
@@ -24,8 +24,8 @@ Run `pod install` after the installation to autolink the Firebase SDK.

See [React Native Firebase](https://rnfirebase.io) and [React Native Firebase Analytics](https://rnfirebase.io/analytics/usage) for more details of Firebase packages.

> info "rnfirebase dependency"
> You will need to follow the [install guide](/docs/connections/sources/catalog/libraries/mobile/react-native/classic) for the rnfirebase dependency too. This may include adding the `GoogleService-Info.plist` and the `google-services.json` file.
> info "Installation"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> info "Installation"
> info "Firebase dependencies"

> info "rnfirebase dependency"
> You will need to follow the [install guide](/docs/connections/sources/catalog/libraries/mobile/react-native/classic) for the rnfirebase dependency too. This may include adding the `GoogleService-Info.plist` and the `google-services.json` file.
> info "Installation"
> You will need to follow the install guide for [Android](https://rnfirebase.io/analytics/usage/installation/android){:target="_blank”} and/or [iOS](https://rnfirebase.io/analytics/usage/installation/ios){:target="_blank”}. This includes adding the `GoogleService-Info.plist` and the `google-services.json` file.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> You will need to follow the install guide for [Android](https://rnfirebase.io/analytics/usage/installation/android){:target="_blank”} and/or [iOS](https://rnfirebase.io/analytics/usage/installation/ios){:target="_blank”}. This includes adding the `GoogleService-Info.plist` and the `google-services.json` file.
> To add the dependencies necessary for Segment's Firebase plugin, you'll need to add the `GoogleService-Info.plist` and the `google-services.json` files to your implementation. For more information, follow the instructions in the [Android](https://rnfirebase.io/analytics/usage/installation/android){:target="_blank”} and/or [iOS](https://rnfirebase.io/analytics/usage/installation/ios){:target="_blank”} documentation.

@forstisabella forstisabella merged commit 94f32be into develop Jun 30, 2025
4 checks passed
@forstisabella forstisabella deleted the PcooperSegment-patch-1 branch June 30, 2025 15:24
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants