-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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): allow disabling redirection on sign in with email #1416
feat(provider): allow disabling redirection on sign in with email #1416
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nextauthjs/next-auth/2YKd2vtxCgriUQyAA2MvA9SCphxB |
cf867d6
to
843f7a8
Compare
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.
Awesome! Finally got a database working locally... 😅 I usually only focus on database-less flows, but I wanted to test this out myself. You have done a nice job, and thanks for the additional documentation!
🎉 This PR is included in version 3.7.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 4.0.0-next.5 🎉 The release is available on: Your semantic-release bot 📦🚀 |
…xtauthjs#1416) * feat: allow to disable client-side redirect for email provider * docs(client): mention that redirect can also be disabled for email provider * feat: only display one email input in email page
What:
In case the user is using the Email provider, make it possible to disable the redirection after the response from the server.
Why:
Example: To make a user signin or signup flow more fluid, it may be desirable to handle email signin without a page reload.
How:
The Email Provider has been added to the list of the providers that allow disabled redirection
canRedirectBeDisabled = isCredentials || isEmail
Checklist:
Also related:
closes #922