Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] [MGT-CHAT] Switching between chats and receiving messages can result in messages being in the wrong chat #3032

Closed
plasne opened this issue Feb 9, 2024 · 0 comments · Fixed by #3278
Assignees
Labels
bug Something isn't working Components: mgt-chat Mgt chat related issues Priority: 1 State: In Review
Milestone

Comments

@plasne
Copy link

plasne commented Feb 9, 2024

Describe the bug
When receiving messages across multiple chats, those messages can get mixed up between the chats both in cache and on-screen.

This is different than #3031, though maybe related, because in this case, the cache starts stable for both chats and so the issue is not graph calls being received and processed incorrectly. Rather this might be a problem with how content is getting cached as it is received or loaded dynamically.
 
To Reproduce
To reproduce, you need a way to switch between chats. You can use ChatList in PR.

Steps to reproduce the behavior:

  1. Switch to chat A in MGT
  2. Switch to chat B in MGT
  3. In Teams, send some messages to chat A and some to chat B
  4. Switch a few times between chat A and chat B in MGT
  5. Take note of the messages, there are likely some that are not in the right chat

NOTE: This is not 100% reproduceable, but is common enough.

Expected behavior
Chat messages sent to chat A are only in chat A.

Screenshots
This is a chat between Joni and Pradeep...
image

Here is between Adele and Joni...
image

Here is Joni and Pradeep in Teams...
image

Environment (please complete the following information):

  • OS: macOS Monterey 12.7.3
  • Browser: edge
  • Framework: react
  • Context: Web
  • Version: v4
  • Provider: Msal2Provider

Additional context
I am a MSFT employee and can be reached by Teams at pelasne.

@plasne plasne added bug Something isn't working Needs: Triage 🔍 labels Feb 9, 2024
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Graph Toolkit Feb 9, 2024
@gavinbarron gavinbarron moved this from Needs Triage 🔍 to Todo 📃 in Graph Toolkit Feb 15, 2024
@gavinbarron gavinbarron added this to the Chat - GA milestone Feb 15, 2024
@Mnickii Mnickii self-assigned this Aug 15, 2024
@Mnickii Mnickii linked a pull request Sep 2, 2024 that will close this issue
6 tasks
@Mnickii Mnickii added Components: mgt-chat Mgt chat related issues Priority: 1 labels Sep 4, 2024
@Mnickii Mnickii modified the milestones: Chat - GA, 4.2.5 Sep 4, 2024
@musale musale closed this as completed Sep 5, 2024
@github-project-automation github-project-automation bot moved this from Todo 📃 to Done ✔️ in Graph Toolkit Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Components: mgt-chat Mgt chat related issues Priority: 1 State: In Review
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants