-
-
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
Add tokenGenerator to Email provider config #537
Comments
@firede Thanks for the submission. This is a great idea! I don't know what we should call the function - maybe something like I can see how, if we implemented this, it would be easy to extend later (e.g. to support things like 2FA input, SMS, etc) which is something a few folks have asked about. In the immediate term, this really nicely solves the problem of people signing in to a browser on a device that does not configured to receive email at the email address they are using (e.g. by checking their phone and typing in the code in the browser on whatever device they are using). Regarding implementation, the verification URLs already have the Provider ID in the URL (e.g. in the With that in mind it would be very easy to update the verification page to handle displaying an input box if an option (e.g. I'm very happy to help with aspect as needed. |
@iaincollins Thanks for the comments.
I voted for
Another option I recommend to use I will try to create a PR for this. |
Hi there! It looks like this issue hasn't had any activity for a while. It will be closed if no further activity occurs. If you think your issue is still relevant, feel free to comment on it to keep ot open. Thanks! |
Hi there! It looks like this issue hasn't had any activity for a while. To keep things tidy, I am going to close this issue for now. If you think your issue is still relevant, just leave a comment and I will reopen it. (Read more at #912) Thanks! |
Please stick to one distinct feature request per issue where possible and raise additional feature quests as separate issues. Try to avoid adding feature requests to existing issues in the comments of issues raised by other users.
Summary of proposed feature
Add
tokenGenerator
to Email provider config. example:Purpose of proposed feature
Customize
notion
style email:Detail about proposed feature
next-auth/src/server/lib/signin/email.js
Lines 12 to 13 in c6d6d9c
Potential problems
Usually need to be used with
sendVerificationRequest
.Describe any alternatives you've considered
A clear and concise description of any alternative options you've considered.Additional context
Any other context, screenshots, etc.Please indicate if you are willing and able to help implement the proposed feature.
Yes! if the idea is accepted.
The text was updated successfully, but these errors were encountered: