Skip to content

[front] Live chat messages #6163

Ben requested to merge feat/live-chat-f6163 into master

Ticket(s) / Related Merge Requests

Summary of Changes

The agreed scope for this card was JUST to update the messages live and have read receipts working with sockets.

There are spin-off issues such as engine#2729 (closed) - please do mention to me anything else that you find that is missing.

Also note there are changes needed before this can work for tenant networks.

Testing Considerations

When user A and user B are both actively watching chat room
  • Messages appear live for both users
  • When you are scrolled far up the message list, user B's chat does NOT jump down to the new message.
When user B is elsewhere on the site
  • User B gets a yellow dot on the Chats section of the site
  • On viewing the chat, the dot disapears.
Feat flag
  • When feat flag is off, socket events aren't listened for or responded to.
Account bleed
  • When switching users, you should NOT receive events for the user you logged out of but SHOULD receive events for the new user that you logged into.

Deployment Considerations

Standard deployment.

Regression Scope

  • Chat messages
  • Sending messages
  • Read receipts

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

Manual, unit.

Screenshots / Screen Recording

When user A and user B are both actively watching chat room

simplescreenrecorder-2024-04-05_19.10.15

When user B is elsewhere on the site

simplescreenrecorder-2024-04-05_19.10.57

Account bleed testing

simplescreenrecorder-2024-04-05_19.12.08

simplescreenrecorder-2024-04-05_19.13.08

Feat flag off

simplescreenrecorder-2024-04-05_19.15.52

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports