KumiSystems/Chatwoot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

chore: Strip unnecessary data from push payload (#2924)

Browse source
This commit is contained in:
Sojan Jose 2021-09-09 13:59:42 +05:30 committed by GitHub
parent 2821777e93
commit 9c4ce9af35
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
app/models/notification.rb
View file

@ -60,6 +60,16 @@ class Notification < ApplicationRecord
}
end
def fcm_push_data
{
id: id,
notification_type: notification_type,
primary_actor_id: primary_actor_id,
primary_actor_type: primary_actor_type,
primary_actor: primary_actor.push_event_data.slice(:conversation_id)
}
end
# TODO: move to a data presenter
def push_message_title
case notification_type

5
app/services/notification/push_notification_service.rb
View file

@ -94,9 +94,10 @@ class Notification::PushNotificationService
{
notification: {
title: notification.notification_type.titleize,
body: notification.push_message_title
body: notification.push_message_title,
sound: 'default'
},
data: { notification: notification.push_event_data.to_json },
data: { notification: notification.fcm_push_data.to_json },
collapse_key: "chatwoot_#{notification.primary_actor_type.downcase}_#{notification.primary_actor_id}"
}
end