From 6b772c9248feae73dc9dd38c6fc5bc3f228097a4 Mon Sep 17 00:00:00 2001
From: Hongbo Wu <hongbo@omnivore.app>
Date: Tue, 22 Nov 2022 10:20:52 +0800
Subject: [PATCH] Disable push notification reminders

---
 packages/api/src/routers/svc/reminders.ts | 28 +++++++++++------------
 packages/rule-handler/src/index.ts        |  2 +-
 2 files changed, 14 insertions(+), 16 deletions(-)

diff --git a/packages/api/src/routers/svc/reminders.ts b/packages/api/src/routers/svc/reminders.ts
index 9536f28ff..5da5dd4da 100644
--- a/packages/api/src/routers/svc/reminders.ts
+++ b/packages/api/src/routers/svc/reminders.ts
@@ -5,8 +5,6 @@ import { kx } from '../../datalayer/knex_config'
 import { setClaims } from '../../datalayer/helpers'
 import { sendEmail } from '../../utils/sendEmail'
 import { env, homePageURL } from '../../env'
-import { sendMulticastPushNotifications } from '../../utils/sendNotification'
-import { getDeviceTokensByUserId } from '../../services/user_device_tokens'
 import { MulticastMessage } from 'firebase-admin/messaging'
 import { UserDeviceToken } from '../../entity/user_device_tokens'
 import { ContentReader } from '../../generated/graphql'
@@ -108,19 +106,19 @@ export function remindersServiceRouter() {
           to: user.email,
         })
 
-        // send push notifications
-        const deviceTokens = await getDeviceTokensByUserId(userId)
-        if (deviceTokens && deviceTokens.length > 0) {
-          const message = messageForPages(pageReminders, deviceTokens)
-          await sendMulticastPushNotifications(userId, message, 'reminder')
-        }
-
-        if (!deviceTokens) {
-          console.log('Device tokens not set:', userId)
-
-          res.status(400).send('Device token Not Found')
-          return
-        }
+        // // send push notifications
+        // const deviceTokens = await getDeviceTokensByUserId(userId)
+        // if (deviceTokens && deviceTokens.length > 0) {
+        //   const message = messageForPages(pageReminders, deviceTokens)
+        //   await sendMulticastPushNotifications(userId, message, 'reminder')
+        // }
+        //
+        // if (!deviceTokens) {
+        //   console.log('Device tokens not set:', userId)
+        //
+        //   res.status(400).send('Device token Not Found')
+        //   return
+        // }
       }
 
       await updateRemindersStatus(models, userId, pagesToUnarchive, remindAt)
diff --git a/packages/rule-handler/src/index.ts b/packages/rule-handler/src/index.ts
index fc5e5533f..6914ce6dd 100644
--- a/packages/rule-handler/src/index.ts
+++ b/packages/rule-handler/src/index.ts
@@ -99,7 +99,7 @@ export const ruleHandler = Sentry.GCPFunction.wrapHttpFunction(
       }
 
       if (type !== EntityType.PAGE) {
-        console.log('Not a page update')
+        console.log('Not a page')
         res.status(200).send('Not Page')
         return
       }