[connectors] Fix timeouts when syncing Zendesk articles (#8843)

* perf: only fetch sections for the category when syncing articles * perf: only fetch the necessary users when syncing articles
dust-tt · Nov 22, 2024 · e415774 · e415774
1 parent b07d2db
commit e415774
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/connectors/src/@types/node-zendesk.d.ts b/connectors/src/@types/node-zendesk.d.ts
@@ -208,6 +208,9 @@ declare module "node-zendesk" {
       };
       sections: {
         list: () => Promise<ZendeskFetchedSection[]>;
+        listByCategory: (
+          categoryId: number
+        ) => Promise<ZendeskFetchedSection[]>;
         show: (
           sectionId: number
         ) => Promise<{ response: Response; result: ZendeskFetchedSection }>;

diff --git a/connectors/src/connectors/zendesk/temporal/activities.ts b/connectors/src/connectors/zendesk/temporal/activities.ts
@@ -331,8 +331,11 @@ export async function syncZendeskArticleBatchActivity({
     `[Zendesk] Processing ${articles.length} articles in batch`
   );
 
-  const sections = await zendeskApiClient.helpcenter.sections.list();
-  const users = await zendeskApiClient.users.list();
+  const sections =
+    await zendeskApiClient.helpcenter.sections.listByCategory(categoryId);
+  const { result: users } = await zendeskApiClient.users.showMany(
+    articles.map((article) => article.author_id)
+  );
 
   await concurrentExecutor(
     articles,