Skip to content

Feat/chat notifications #2711

Fausto Arcidiacono requested to merge feat/chat-notifications-e2711 into master

Ticket(s) / Related Merge Requests

#2711

Summary of Changes

Added push notifications settings and notification event subscriptions for chat rooms

Testing Considerations

  • Off toggle sticks when muting a chat.
  • On toggle sticks when enabling notifications for a chat.
  • Rooms can be created.
  • I can see the list of room members.
  • Notifications are received when toggle is on.

Deployment Considerations

Merge new helm charts changes (helm-charts!168 (merged)) before merging the engine/frontend.

Add new db table in Vitess:

CREATE TABLE IF NOT EXISTS minds_chat_room_member_settings
(
    tenant_id int,
    room_guid bigint,
    member_guid bigint,
    notifications_status enum('MUTED', 'MENTIONS', 'ALL'),
    PRIMARY KEY (tenant_id, room_guid, member_guid),
    INDEX (tenant_id, room_guid),
    INDEX (member_guid)
);

Run the following query to fix the existing rooms created prior to this MR:

INSERT INTO minds_chat_room_member_settings
    (
        tenant_id,
        room_guid,
        member_guid,
        notifications_status
    )
SELECT
    tenant_id,
    room_guid,
    member_guid,
    "ALL"
FROM
    minds_chat_members
ON DUPLICATE KEY UPDATE
    notifications_status = "ALL";

Regression Scope

Chats

Platform Affected (web, mobile, etc)

Engine for Chats.

Developer Testing Completed

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

Screenshots / Screen Recording

Off / on toggle

simplescreenrecorder-2024-04-15_11.03.55

Rooms can be created.

simplescreenrecorder-2024-04-15_12.28.49

I can see the list of room members.

image

Notifications are received

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 Fausto Arcidiacono

Merge request reports