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

fix(firebase): add ability to pass in firebase config #5055

Closed
wants to merge 1 commit into from
Closed

fix(firebase): add ability to pass in firebase config #5055

wants to merge 1 commit into from

Conversation

chanceaclark
Copy link
Contributor

@chanceaclark chanceaclark commented Jul 31, 2022

☕️ Reasoning

Adds the ability to pass in Firebase config into the adapter to initialize the app, rather than initializing by default. This is useful if you've initialized the app on the client and want to reuse the config on the server. Alongside that, adds some documentation on how to use it.

Note: Ran prettier --write packages/adapter-firebase which cleaned up some files.

🧢 Checklist

  • Documentation
  • Tests
  • Ready to be merged

🎫 Affected issues

Fixes: #4927

📌 Resources

@vercel
Copy link

vercel bot commented Jul 31, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
next-auth ✅ Ready (Inspect) Visit Preview Sep 2, 2022 at 6:54PM (UTC)

@github-actions github-actions bot added adapters Changes related to the core code concerning database adapters firebase @auth/firebase-adapter labels Jul 31, 2022
@vercel vercel bot temporarily deployed to Preview July 31, 2022 17:43 Inactive
@vercel vercel bot temporarily deployed to Preview July 31, 2022 17:47 Inactive
@chanceaclark chanceaclark marked this pull request as ready for review July 31, 2022 17:50
@chanceaclark
Copy link
Contributor Author

cc. @balazsorban44

@itsmylife
Copy link

@chanceaclark @balazsorban44 @ndom91 @lluia Sorry for pinging you all but will we have approval soon?

@vercel vercel bot temporarily deployed to Preview September 2, 2022 18:54 Inactive
balazsorban44 pushed a commit that referenced this pull request Feb 5, 2023
Co-authored-by: Balázs Orbán <info@balazsorban.com>
fixes #5049
closes #6230
closes #5449
closes #5055
fixes #4927

BREAKING CHANGE:
The adapter now expects `firebase-admin` instead of the `firebase` package and also supports either passing a config object or a firestore instance.
balazsorban44 pushed a commit that referenced this pull request Feb 5, 2023
Co-authored-by: Balázs Orbán <info@balazsorban.com>
fixes #5049
closes #6230
closes #5449
closes #5055
fixes #4927

BREAKING CHANGE:
The adapter now expects `firebase-admin` instead of the `firebase` package and also supports either passing a config object or a firestore instance.
notrab pushed a commit to notrab/next-auth that referenced this pull request Feb 22, 2023
…hjs#6225)

Co-authored-by: Balázs Orbán <info@balazsorban.com>
fixes nextauthjs#5049
closes nextauthjs#6230
closes nextauthjs#5449
closes nextauthjs#5055
fixes nextauthjs#4927

BREAKING CHANGE:
The adapter now expects `firebase-admin` instead of the `firebase` package and also supports either passing a config object or a firestore instance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adapters Changes related to the core code concerning database adapters firebase @auth/firebase-adapter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Firebase Adapter Fails with App Name Already exist error
2 participants