Skip to content
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

Allow Android apps to use initializeApp() to configure firebase database #1193

Merged
merged 1 commit into from
Jun 25, 2018

Conversation

caseyt
Copy link

@caseyt caseyt commented Jun 8, 2018

Updated RNFirebaseDatabase.java to use appName if provided.

To test:
Create android app without google-services.json and call initializeApp() in JS code.

@Salakar Salakar self-requested a review June 25, 2018 15:57
@Salakar Salakar self-assigned this Jun 25, 2018
@Salakar Salakar added type: bug New bug report platform: android plugin: database Firebase Realtime Database labels Jun 25, 2018
@Salakar Salakar added this to the v4.3.0 Release milestone Jun 25, 2018
@Salakar
Copy link
Member

Salakar commented Jun 25, 2018

@caseyt thanks for this, looks like iOS needs doing as well, will merge this one in for now and make a note on #1160 re: iOS

@Salakar Salakar merged commit c291623 into invertase:master Jun 25, 2018
@Salakar Salakar removed this from the v4.3.0 Release milestone Jul 1, 2018
@Salakar
Copy link
Member

Salakar commented Jul 10, 2018

@caseyt thanks again for this, this is now live in the v4.3.0 release.


Loving react-native-firebase and the support we provide? Please consider supporting us with any of the below:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: android plugin: database Firebase Realtime Database type: bug New bug report
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants