Skip to content

changed password routing and connected fe to be#356

Closed
yumi520 wants to merge 1 commit intomainfrom
348/change-password-route
Closed

changed password routing and connected fe to be#356
yumi520 wants to merge 1 commit intomainfrom
348/change-password-route

Conversation

@yumi520
Copy link
Contributor

@yumi520 yumi520 commented Mar 13, 2026

ℹ️ Issue #348

Closes #348

📝 Description

Briefly list the changes made to the code:

  1. Added methods to change password
  2. Error handling if passwords aren't meeting requirements

Test Changes

  • Tested if password doesn't meet requirement by using the same password
Screenshot 2026-03-13 at 11 17 47 AM
  • Tested that the new password change worked (the old password no longer works)
Screenshot 2026-03-13 at 11 18 30 AM

@prooflesben prooflesben requested a review from adityapat24 March 13, 2026 19:38
Copy link
Contributor

@adityapat24 adityapat24 left a comment

Choose a reason for hiding this comment

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

Looks good! @prooflesben I think we can go with this implementation instead of what I did

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.

DEV - Change Password Backend Route/Connect to Change Password Frontend to Backend

3 participants