Skip to content

feat: implement organization password update functionality #377

Open
AliRana30 wants to merge 1 commit into
c2siorg:developfrom
AliRana30:feat/password-update-317
Open

feat: implement organization password update functionality #377
AliRana30 wants to merge 1 commit into
c2siorg:developfrom
AliRana30:feat/password-update-317

Conversation

@AliRana30
Copy link
Copy Markdown

Description

Transformed the static "Passwords" page in the Organization settings into a fully functional feature. Previously, the page provided the UI shell but lacked the logic to handle password changes.

Technical Implementation:

  • Added useState for managing input fields: oldPassword, newPassword, and confirmPassword.
  • Implemented client-side validation, ensuring all fields are filled, new passwords match, and meet the minimum character requirement (6 characters for Firebase Auth).
  • Integrated the Firebase reauthenticateWithCredential logic, which is a security requirement for password updates in Firebase.
  • Used user.updatePassword to securely persist the changes in the authentication backend.
  • Added a Snackbar with a Material-UI Alert to provide success and error notifications, and a CircularProgress indicator to give visual feedback during the update process.

Related Issue

Refers to issue #317

Motivation and Context

This update completes the security settings for organizations. It solves the issue where users were presented with a non-functional form, ensuring that organizations can manage their account security directly from the dashboard.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • Functional testing performed locally using Firebase Auth.

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.

1 participant