fix: Append 'category' and 'isRead' as query params in bulk update API

Author: reshmarevi17

src/Siren.ts

@@ -242,7 +242,7 @@ export class Siren {
       };
 
       if(category) params.category = category;
-
+      
       response = await notificationsBulkUpdates(
         this.token,
         this.recipientId,
@@ -299,6 +299,7 @@ export class Siren {
 
       if (isRead !== undefined) params.isRead = isRead;
       if(category) params.category = category;
+      
       response = await notificationsBulkUpdates(
         this.token,
         this.recipientId,

src/api/notificationsBulkUpdates.ts

@@ -12,20 +12,29 @@ const notificationsBulkUpdates = (
   inAppRecipientId: string,
   data: {
     until: string;
-    isRead?: boolean;
     operation: BulkUpdateType;
+    category?: string,
+    isRead?: boolean
   }
 ): Promise<ActionResponse> => {
-  const path = `api/v2/in-app/recipients/${inAppRecipientId}/notifications/bulk-update`;
+  let path = `api/v2/in-app/recipients/${inAppRecipientId}/notifications/bulk-update`;
 
+  const { category, isRead, ...filteredData } = data;
+
+  if (category) 
+    path = `${path}?category=${category}`;
+  
+  if (isRead) 
+    path = path.includes('?') ? `${path}&isRead=${isRead}` : `${path}?isRead=${isRead}`;
+  
   const resp: Promise<ActionResponse> = httpPost(
     {
       path,
       token,
       errorLevel: ErrorLevelType.ERROR,
       operation: operations[data.operation ?? BulkUpdateType.MARK_AS_READ]
     },
-    data
+    filteredData
   );
 
   return resp;