Skip to content

chore: Prevent GoogleService-info.plist from being double copied#225

Merged
mahmoud-elmorabea merged 3 commits into
feature/fcm-supportfrom
google-services-plist-file-conflict
Mar 13, 2025
Merged

chore: Prevent GoogleService-info.plist from being double copied#225
mahmoud-elmorabea merged 3 commits into
feature/fcm-supportfrom
google-services-plist-file-conflict

Conversation

@mahmoud-elmorabea

@mahmoud-elmorabea mahmoud-elmorabea commented Mar 10, 2025

Copy link
Copy Markdown
Contributor

The ReactNative Firebase dependency requires GoogleService-info.plist to be added to the iOS project.

RN Firebase doesn't check if the file already exists before adding it to the project. Since we cannot depend on customers having ReactNative Firebase to copy the file. Our plugin copies the file.

This PR adds a check at the path where ReactNative Firebase copies the file first before copying it.

@mahmoud-elmorabea mahmoud-elmorabea self-assigned this Mar 10, 2025
@mahmoud-elmorabea mahmoud-elmorabea marked this pull request as ready for review March 11, 2025 00:06
@mahmoud-elmorabea mahmoud-elmorabea requested a review from a team as a code owner March 11, 2025 00:06

@Shahroz16 Shahroz16 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

After a team discussion, we are going forward with this approach until we find a better one.

@mahmoud-elmorabea mahmoud-elmorabea requested review from a team and Shahroz16 March 12, 2025 14:20

@Shahroz16 Shahroz16 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm, i have created an optional PR where i check for xcode reference as well, incase we want to add it.

@mahmoud-elmorabea mahmoud-elmorabea requested a review from a team March 12, 2025 19:06
Comment thread plugin/src/ios/withGoogleServicesJsonFile.ts Outdated
Comment thread plugin/src/ios/withGoogleServicesJsonFile.ts Outdated
Comment thread __tests__/ios/fcm/GoogleService-InfoCopied.test.js Outdated
@mahmoud-elmorabea mahmoud-elmorabea merged commit c0e51f0 into feature/fcm-support Mar 13, 2025
@mahmoud-elmorabea mahmoud-elmorabea deleted the google-services-plist-file-conflict branch March 13, 2025 10:08
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