-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
feat(provider): require to import every provider individually #2518
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nextauthjs/next-auth/DJZ1RsXQV8sJ2zD8oC3yh7PU5XCg |
Codecov Report
@@ Coverage Diff @@
## next #2518 +/- ##
=======================================
Coverage 11.56% 11.56%
=======================================
Files 85 85
Lines 1314 1314
Branches 370 370
=======================================
Hits 152 152
Misses 966 966
Partials 196 196 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We did pair review on this with @balazsorban44 , looking good 💪🏽 💯
…thjs#2518) Adds a new way to import providers for modularity and better tree-shaking. BREAKING CHANGE: Providers now have to be imported one-by-one: Example: ```diff - import Provider from "next-auth/providers" - Providers.Auth0({...}) + import Auth0Provider from "next-auth/providers/auth0" + Auth0Provider({...}) ```
Fixes #2515
With this PR, the newly added provider also won't have to add anything manually to the
providers.d.ts
file to be included. We generate it from the provider file.BREAKING CHANGE:
Providers now have to be imported one-by-one:
Example: