fix: Remove unnecessary attributes from notification deleted action c…

…able event payload (chatwoot#9295) fix: simplify notification deleted payload event
CarlosPedroGervasioJunior · Apr 25, 2024 · 1bf3ad9 · 1bf3ad9
1 parent 9086650
commit 1bf3ad9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/listeners/action_cable_listener.rb b/app/listeners/action_cable_listener.rb
@@ -18,7 +18,7 @@ def notification_deleted(event)
 
     notification, account, unread_count, count = extract_notification_and_account(event)
     tokens = [event.data[:notification].user.pubsub_token]
-    broadcast(account, tokens, NOTIFICATION_DELETED, { notification: notification.push_event_data, unread_count: unread_count, count: count })
+    broadcast(account, tokens, NOTIFICATION_DELETED, { notification: { id: notification.id }, unread_count: unread_count, count: count })
   end
 
   def account_cache_invalidated(event)

diff --git a/spec/listeners/action_cable_listener_spec.rb b/spec/listeners/action_cable_listener_spec.rb
@@ -142,7 +142,9 @@
         'notification.deleted',
         {
           account_id: notification.account_id,
-          notification: notification.push_event_data,
+          notification: {
+            id: notification.id
+          },
           unread_count: 1,
           count: 1
         }