[front] Live chat messages #6163
Ticket(s) / Related Merge Requests
- Issue card: #6163 (closed)
- Engine MR: engine!1640 (merged)
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