microsoft-graph-toolkit icon indicating copy to clipboard operation
microsoft-graph-toolkit copied to clipboard

[BUG][MGT-CHAT] The profile card in the Chat component will get into an invalid state if any navigation is done using the Organization section

Open MattBillFred opened this issue 4 months ago • 1 comments

Describe the bug When using the chat component, received messages will show a profile icon of the user you are communication with. Clicking on this profile icon will open a profile card containing details about that user. There is a section named "Organization" that will let users click on profiles of other users in the organization. This action will then show that user's profile information.

The problem is that the profile picture will quickly revert back to the original user, leaving the profile information in a strange state.

Additionally, the Quick Chat box will send a message to the original user, and not to the user who's name is showing in the profile.

To Reproduce

  1. Go to the MGT Chat Test Harness.
  2. Click on the profile icon next to a received message.
  3. Within the profile card, click on the Organization chart icon.
  4. Select a user other than the original user.
  5. Watch the profile card be update with the selected user's information.
  6. Moments later, watch as the profile picture is reverted back.
  7. Send a quick message from within the profile card.
  8. Watch as the message goes to the original user, and not to the newly selected user.

Expected behavior When navigating the organization the profile card, the profile icon and the quick message recipient to match the details of the newly selected user.

Screenshots Before clicking image

Immediately after clicking image

A few seconds after clicking image

Environment (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome
  • Framework: React
  • Context: Web
  • Version: MGT 4
  • Provider: Msal2Provider

Additional context

MattBillFred avatar Feb 28 '24 15:02 MattBillFred

I wonder if this is related to the presence refresh?

This needs to be tested outside of the chat component to validate the scope of impact

gavinbarron avatar Feb 28 '24 16:02 gavinbarron