@financial-times/cp-content-pipeline-schema 2.11.1 → 2.12.0

package/src/model/schemas/capi/placeholder.ts

@@ -23,6 +23,7 @@ const placeholderMetadataSchema = baseMetadataSchema.pick({
   standout: true,
   publishedDate: true,
   firstPublishedDate: true,
+  publishReference: true,
   accessLevel: true,
   editorialDesk: true,
   canBeSyndicated: true,

package/src/resolvers/content-tree/references/Video.ts

@@ -15,13 +15,18 @@ export const Video = {
       return capiResponse.title()
     } catch (error) {
       if (error instanceof Error) {
-        throw new OperationalError({
-          code: 'VIDEO_CONTENT_FETCH_ERROR',
-          message: `Error getting content object for video ${parent.reference.id}`,
-          cause: error,
+        context.logger.warn({
+          event: 'RECOVERABLE_ERROR',
+          error: new OperationalError({
+            code: 'VIDEO_CONTENT_FETCH_ERROR',
+            message: `Error getting content object for video ${
+              parent.reference.id
+            } in article ${parent.contentApiData?.id()}`,
+            cause: error,
+          }),
         })
+        return 'Sorry, we are unable to find this video'
       }
-
       throw error
     }
   },

package/src/resolvers/content.ts

@@ -29,6 +29,7 @@ const contentResolvers: ContentResolvers = {
   design: (parent) => parent.design(),
   editorialDesk: (parent) => parent.editorialDesk(),
   firstPublishedDate: (parent) => parent.firstPublishedDate(),
+  publishReference: (parent) => parent.publishReference(),
   id: (parent) => parent.id(),
   instantAlertConcept: (parent) => parent.instantAlertConcept(),
   mainImage: (parent) => parent.mainImage(),