Handle topic cover info

battis · Dec 26, 2024 · e519cf4 · e519cf4
1 parent 6a63235
commit e519cf4
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 10 deletions.
diff --git a/packages/datadirect/src/api/DataDirect/common/prepareContent.ts b/packages/datadirect/src/api/DataDirect/common/prepareContent.ts
@@ -36,6 +36,9 @@ export function prepareContent(
     case 'Cover Brief':
     case 'Cover Image':
     case 'Cover Title':
+      throw new Error(
+        `All content for ${contentType.Content} is captured by /api/topiccontentget/:TopicID`
+      );
     case 'Roster':
     case 'Learning Tool':
       throw new Error(`Capturing ${contentType.Content} is not yet supported`);

diff --git a/packages/myschoolapp-reporting/src/workflows/Snapshot/Topics.ts b/packages/myschoolapp-reporting/src/workflows/Snapshot/Topics.ts
@@ -78,16 +78,22 @@ export async function capture(
             )
           });
         } catch (error) {
-          Content?.push({
-            ...item,
-            ObjectType,
-            Content: { error }
-          });
-          cli.log.error(
-            `Error capturing Topic ${TopicID} content of type ${
-              ObjectType?.Name
-            } for group ${Id}: ${cli.colors.error(error)}`
-          );
+          if (
+            `${error}`.endsWith('is captured by /api/topiccontentget/:TopicID')
+          ) {
+            Content.push({ ...item, ObjectType });
+          } else {
+            Content?.push({
+              ...item,
+              ObjectType,
+              Content: { error }
+            });
+            cli.log.error(
+              `Error capturing Topic ${TopicID} content of type ${
+                ObjectType?.Name
+              } for group ${Id}: ${cli.colors.error(error)}`
+            );
+          }
         }
       }
       Topics.push({ ...topic, Content: Content.length ? Content : undefined });