feat(push-notifications): delete expired tokens for one-signal provid…

…er (#1088)
ConduitPlatform · Jul 2, 2024 · e88211f · e88211f
1 parent f82aa42
commit e88211f
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/modules/push-notifications/src/providers/OneSignal.provider.ts b/modules/push-notifications/src/providers/OneSignal.provider.ts
@@ -6,6 +6,7 @@ import {
 } from '../interfaces/ISendNotification.js';
 import { createConfiguration, DefaultApi, Notification } from '@onesignal/node-onesignal';
 import ConduitGrpcSdk from '@conduitplatform/grpc-sdk';
+import { NotificationToken } from '../models/index.js';
 
 export class OneSignalProvider extends BaseNotificationProvider<IOneSignalSettings> {
   private client?: DefaultApi;
@@ -42,7 +43,12 @@ export class OneSignalProvider extends BaseNotificationProvider<IOneSignalSettin
         include_player_ids: Array.isArray(token) ? token : [token],
       };
     }
-    await this.client!.createNotification(notification);
+    const response = await this.client!.createNotification(notification);
+    if (response.errors?.invalid_player_ids?.length) {
+      await NotificationToken.getInstance().deleteMany({
+        token: { $in: response.errors.invalid_player_ids },
+      });
+    }
     return;
   }