search

Saas-Starter-Kit / Saas-Kit-prisma

🚀A template for building Software-as-Service (SAAS) apps with Reactjs, Nextjs, Prisma and OpenAI integration
https://www.saasstarterkit.com/dashboard/test243/main
439 stars 71 forks source link

improved styling of verification email #2

Closed alawi2306 closed 9 months ago

alawi2306 commented 9 months ago

Made the verification email more aesthetic .

iqbal125 commented 9 months ago

Hey thanks for opening a pull request appreciate it. For emails we use the text because of the automated Playwright tests. I believe the tests will no longer work with this update.

alawi2306 commented 9 months ago

Okay cool. I have a question about the login functionality. The verification of the email seems great for the register part, but making the user check their email to login for that part seems a little much. I was wondering if we could work on a login functionality using bcrypt to hash the user's password in the db then comparing it for the login . Problem is I'm new to next-auth and it's a nightmare trying to create a new callback for this.

Wondering if we could work on this or are you okay with keeping it the same?

iqbal125 commented 9 months ago

That's unfortunately a big issue in next-auth. They dont support passwords at all. We have to implement it from scratch, which can be very error prone.

Im thinking about switching to Lucia auth once it becomes more mature.

If you did just want to add it for your project, here is a good guide for it. https://nextjs.org/learn/dashboard-app/adding-authentication