Add a way to reset password#38
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds password reset functionality to the application, allowing users to request a password reset link via email. The implementation includes email domain validation, user existence verification, and a dedicated UI page for initiating the reset process.
Key changes:
- New
/reset-passwordpage with form UI for requesting password reset links - Server action implementing email validation, domain checking, and user verification before sending reset emails
- Integration of "Forgot password?" link on the login page
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| app/reset-password/page.tsx | New client component providing the password reset request UI with form validation and state management |
| app/reset-password/actions.ts | Server action handling password reset requests with email domain validation and user existence checks |
| app/login/page.tsx | Added "Forgot password?" link directing to the reset password page |
| utils/supabase/middleware.ts | Excluded /reset-password route from authentication requirement |
| .env.development | Updated allowed email domain to marmelab.com |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
polish: it's a security issue to send a feedback if the user email does not exist. Prefer to send a generic feedback like the second you display |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Capture.video.du.2025-10-22.14-46-14.mp4