Conversation
✅ Deploy Preview for paisable canceled.
|
|
Thank you for opening the PR, @Adarsh-ops. Kindly pull the latest changes from master and then resolve the conflicts that are occuring. |
There was a problem hiding this comment.
Line 5 of your commit: const sendEmail=require('../utils/sendEmail'); is not correct as this would lead the compiler to try to find a file named sendEmail, which does not exist. You should import the function sendEmail from utils.js instead of trying to search for a file. Also, there is some kind of route mismatch between the frontend and the backend which is causing a 404 error. Check for that too. Kindly make the respective changes.
|
Also, it would be awesome if you could add the Paisable text/logo at the top of the forgot password card so that users can return to the homepage |
|
Yeah got it, will work on it |
|
Also, resolve the conflicts. |
|
This should work @archa8 |
|
It is still not working somehow on my end. Could you please once contact me on Discord and walk me through it? |
|
That shouldn't have happened, could you please specify what exactly is showing on your end @archa8 |
|
Have made some changes, should work now |
|
@Adarsh-ops, it still shows |
|
Yes you need to add a .env file with these environment variables with it |
Great! I've set these up using Brevo into my .env |
I added a forgot-password feature which asks the user's email and sends the reset-password link to email and the user is able to update the password.
Description
If the user has forgotten his password, he clicks on the forgot password button and is redirected to the forgot password page where the user fills in his email, clicking on "Send Reset Link", sends user a reset link to /reset-password endpoint. Coming with a valid token, resets user's password else throws an error.
Related Issue
Fixes #116
Motivation and Context
This change was required because if the user by chance forgets the password, he won't be able to login again, he would need to create a new account.
Types of Changes
How Has This Been Tested?
This has been tested by running it locally on my machine and putting the respective environment variables
Screenshots (if applicable):
Checklist