Merge pull request #43 from olusoladavid/ch-refactor-reminder-160126500

#160126500 Include message payload in push notifications
olusoladavid · Aug 29, 2018 · a306c19 · a306c19
2 parents f4b126d + 39c044e
commit a306c19
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/server/workers/sendReminder.js b/server/workers/sendReminder.js
@@ -26,12 +26,16 @@ const processNotifications = async () => {
     users.forEach(async (user) => {
       if (user.reminderisset) {
         try {
-          const msg = '{ type: reminder }';
+          const msg = JSON.stringify({
+            title: 'What\'s on your mind today?',
+            body: 'Time to update your diary',
+            tag: 'reminder',
+          });
           const options = { TTL: 86400 };
           const pushSub = JSON.parse(user.push_sub);
           await webpush.sendNotification(pushSub, msg, options);
         } catch (err) {
-          console.error(`Could not send reminder for user: ${user.email}`);
+          console.error(`Could not send reminder for user: ${user.email} due to ${err.message}`);
         }
       }
     });