fix: update password when adding existing user to new org #2

JoeBot · 2026-03-08T19:50:19-04:00

JoeBot commented

2026-03-08 19:50:19 -04:00

Summary

When an existing user was added to a new organization via the Manage Members UI, the password entered on the form was silently ignored. The user's password remained whatever it was set to when originally created in a different org.
This caused the new password to not work at login, with no indication to the admin that the password wasn't applied.
Fix: when addMember finds an existing user by email, it now updates their password_hash with the newly provided password before creating the membership.

Root Cause

In organizations.service.ts addMember(), the existing-user branch (lines 141-155) only created the org membership — it never touched password_hash. The password was only set in the else branch when creating a brand new user.

Test plan

Add an existing user (who belongs to Org A) to Org B via Manage Members, entering a new password
Verify the user can log in with the new password
Verify adding a brand-new user still works as before
Verify re-activating an inactive membership still works

## Summary - When an existing user was added to a new organization via the Manage Members UI, the password entered on the form was **silently ignored**. The user's password remained whatever it was set to when originally created in a different org. - This caused the new password to not work at login, with no indication to the admin that the password wasn't applied. - Fix: when `addMember` finds an existing user by email, it now updates their `password_hash` with the newly provided password before creating the membership. ## Root Cause In `organizations.service.ts` `addMember()`, the existing-user branch (lines 141-155) only created the org membership — it never touched `password_hash`. The password was only set in the `else` branch when creating a brand new user. ## Test plan - [ ] Add an existing user (who belongs to Org A) to Org B via Manage Members, entering a new password - [ ] Verify the user can log in with the new password - [ ] Verify adding a brand-new user still works as before - [ ] Verify re-activating an inactive membership still works

JoeBot added 1 commit 2026-03-08 19:50:20 -04:00

fix: update password when adding existing user to new org 3bf6b8c6c9

When an existing user was added to a new organization via the member
management UI, the password entered in the form was silently ignored.
This caused the user to be unable to log in with the password they
were given, since the hash in the database was from their original
account creation for a different org.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

JoeBot closed this pull request

2026-03-17 07:47:38 -04:00

JoeBot deleted branch fix/add-member-password-ignored

2026-04-02 12:31:46 -04:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: JoeBot/HOA_Financial_Platform#2