fix(web_push_notifications): Send message with welcome notifications (#4471)

* fix(web_push_notifications): Send grouping title in welcome notification

* fix(web_push_notifications): Read message from last notification
lolsob-rspec
Sorin Davidoi 2017-08-01 06:00:13 +02:00 committed by Eugen Rochko
parent d467afff43
commit d8f78827a4
2 changed files with 3 additions and 2 deletions

View File

@ -9,7 +9,7 @@ const notify = options =>
if (notifications.length === MAX_NOTIFICATIONS) {
// Reached the maximum number of notifications, proceed with grouping
const group = {
title: formatGroupTitle(notifications[0].data.message, notifications.length + 1),
title: formatGroupTitle(options.data.message, notifications.length + 1),
body: notifications
.sort((n1, n2) => n1.timestamp < n2.timestamp)
.map(notification => notification.title).join('\n'),
@ -19,7 +19,7 @@ const notify = options =>
data: {
url: (new URL('/web/notifications', self.location)).href,
count: notifications.length + 1,
message: notifications[0].data.message,
message: options.data.message,
},
};

View File

@ -161,6 +161,7 @@ class Web::PushSubscription < ApplicationRecord
content: translate('push_notifications.subscribed.body'),
actions: [],
url: web_url('notifications'),
message: translate('push_notifications.group.title'), # Do not pass count, will be formatted in the ServiceWorker
}
),
endpoint: endpoint,