Skip to content

Conversation

@Adarsh-ops
Copy link

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (improvement to an existing feature)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Test (adds or updates tests only)
  • Documentation (non-code change)

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):

image image

Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

@netlify
Copy link

netlify bot commented Oct 19, 2025

Deploy Preview for paisable canceled.

Name Link
🔨 Latest commit 94d7785
🔍 Latest deploy log https://app.netlify.com/projects/paisable/deploys/68fb8e949e2a5800089883ac

@archa8
Copy link
Member

archa8 commented Oct 20, 2025

Thank you for opening the PR, @Adarsh-ops. Kindly pull the latest changes from master and then resolve the conflicts that are occuring.

Copy link
Member

@archa8 archa8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@archa8
Copy link
Member

archa8 commented Oct 22, 2025

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

@Adarsh-ops
Copy link
Author

Yeah got it, will work on it

@archa8
Copy link
Member

archa8 commented Oct 22, 2025

Also, resolve the conflicts.

@Adarsh-ops
Copy link
Author

This should work @archa8

@archa8
Copy link
Member

archa8 commented Oct 23, 2025

It is still not working somehow on my end. Could you please once contact me on Discord and walk me through it?

@Adarsh-ops
Copy link
Author

That shouldn't have happened, could you please specify what exactly is showing on your end @archa8

@Adarsh-ops
Copy link
Author

Have made some changes, should work now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Add 'Forgot Password' feature

2 participants