Skip to content

Conversation

@fernando-cortez
Copy link
Collaborator

@fernando-cortez fernando-cortez commented Jul 3, 2025

Description

This PR provides a fix for an error raised when changing profiles. The current LocalSessionUser needed to be added to the local session before attempting to remove when reseting a session. It has since been added to the initial login flow and whenever a profile is changed by the user.

Issue Number(s)

MTT-11948

Contribution checklist

  • [ N/A ] Tests have been added for boss room and/or utilities pack
  • [ N/A ] Release notes have been added to the project changelog file and/or package changelog file
  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • JIRA ticket ID is in the PR title or at least one commit message
  • Include the ticket ID number within the body message of the PR to create a hyperlink
  • [ N/A ] An Index entry has been added in readme.md if applicable

@unity-cla-assistant
Copy link

unity-cla-assistant commented Jul 3, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ fernando-cortez
❌ Fernando Cortez


Fernando Cortez seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@fernando-cortez fernando-cortez marked this pull request as ready for review July 3, 2025 20:47
@fernando-cortez fernando-cortez requested a review from a team as a code owner July 3, 2025 20:47
Copy link
Contributor

@Elfi0Kuhndorf Elfi0Kuhndorf left a comment

Choose a reason for hiding this comment

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

Works and LGTM🥳

@fernando-cortez fernando-cortez merged commit 953ed0e into develop Jul 10, 2025
15 of 16 checks passed
@fernando-cortez fernando-cortez deleted the fix/reset-local-user branch July 10, 2025 17:20
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.

3 participants