Slack: Add logs to understand why some Slack thread in datasource doc…

…uemtnts have no sourceUrl (#4545)
dust-tt · Apr 3, 2024 · c6c6cb6 · c6c6cb6
1 parent fa8b020
commit c6c6cb6
Showing 1 changed file with 23 additions and 0 deletions.
diff --git a/connectors/src/connectors/slack/temporal/activities.ts b/connectors/src/connectors/slack/temporal/activities.ts
@@ -445,6 +445,17 @@ export async function syncNonThreaded(
     });
     if (linkRes.ok && linkRes.permalink) {
       sourceUrl = linkRes.permalink;
+    } else {
+      logger.error(
+        {
+          connectorId,
+          channelId,
+          channelName,
+          messageTs: firstMessage.ts,
+          linkRes,
+        },
+        "No documentUrl for Slack non threaded: Failed to get permalink"
+      );
     }
   }
   const lastMessage = messages.at(-1);
@@ -593,6 +604,18 @@ export async function syncThread(
     });
     if (linkRes.ok && linkRes.permalink) {
       sourceUrl = linkRes.permalink;
+    } else {
+      logger.error(
+        {
+          connectorId,
+          channelId,
+          channelName,
+          threadTs,
+          messageTs: firstMessage.ts,
+          linkRes,
+        },
+        "No documentUrl for Slack thread: Failed to get permalink"
+      );
     }
   }
   const lastMessage = allMessages.at(-1);