Feat/chat notifications #2711
Ticket(s) / Related Merge Requests
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.
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