Skip to content

2626 Fix user profile update - #3462

Draft
arshiamasih wants to merge 7 commits into
developfrom
fix-user-profile-update
Draft

2626 Fix user profile update#3462
arshiamasih wants to merge 7 commits into
developfrom
fix-user-profile-update

Conversation

@arshiamasih

Copy link
Copy Markdown
Member

What changes did you make?

  • When updating account, the email verification flow is triggered only when the email is changed
    • The email verification flow requires the user to confirm email from their updated email address, forces a log out and requires the user to log in with their newly updated email
  • If only the name is change (first, last, both) a success message is displayed and the user is not logged out

Why did you make the changes (we will use this info to test)?

  • The User Profile update feature should not require email verification if the email is not changed, but should send an email to verify the new email if the change involves modifying the login email, in order for the feature to work properly.

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

[BEFORE] Updating only the names and not the email address triggers the email verification flow.

image
image

Visuals after changes are applied

[AFTER NAME ONLY] Updating only the Name(s) displays a success message and does not trigger the email verification flow.

  • The name updates immediately in the header.
  • The user is not logged out.

image

[AFTER WITH EMAIL] Updating the Email only OR Email and Name(s) triggers the email verification flow:

Email change only:

image
image

Email + Name change:

image
image

Comment on lines +170 to +178
user: {
id: updatedUser.id,
firstName: updatedUser.firstName,
lastName: updatedUser.lastName,
email: updatedUser.email,
isAdmin: updatedUser.isAdmin,
emailConfirmed: updatedUser.emailConfirmed,
isSecurityAdmin: updatedUser.isSecurityAdmin
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@entrotech hmm maybe it is overkill to send the entire user what do you think? We need to update userContext on front end when updating names only since we don't force a log out.

@arshiamasih
arshiamasih marked this pull request as draft August 25, 2026 22:15
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: Review and Fix User Profile Update Feature

1 participant