Skip to content

[front] Message deletion via sockets #6179

Ben requested to merge feat/sock-msg-del-f6179 into master

Ticket(s) / Related Merge Requests

Summary of Changes

Moving message deletion to socket events, such that messages are deleted live for other users in the chat.

Testing Considerations

  • Messages delete live for yourself - on sending a new message or reloading it does not reappear.
  • Messages delete live for other users in the room - on sending a new message or reloading it does not reappear.
  • Messages delete for all users in a group chat live.
  • Room list message preview updates live on message deletion for both users.

Deployment Considerations

Standard deployment.

Ping @msantang78 on merge so that he can plug into this event on the mobile side.

Regression Scope

  • Chat room message list.
  • Message previews in rooms list.

Platform Affected (web, mobile, etc)

Web.

Developer Testing Completed

Manual, unit (jasmine/karma), unit (phpspec).

Screenshots / Screen Recording

Messages delete live for yourself and other users

simplescreenrecorder-2024-05-13_16.40.08

Messages delete for all users in a group chat live.

simplescreenrecorder-2024-05-13_16.40.48

Room list message preview updates live on message deletion for both users.

simplescreenrecorder-2024-05-13_16.41.15

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