Merge pull request #26 from THEOplayer/yospace-ad-integration

Integrate Yospace connector with custom server-side ad integration API

.changeset/config.json

@@ -7,5 +7,8 @@
   "access": "public",
   "baseBranch": "main",
   "updateInternalDependencies": "minor",
-  "ignore": []
+  "ignore": [],
+  "___experimentalUnsafeOptions_WILL_CHANGE_IN_PATCH": {
+    "onlyUpdatePeerDependentsWhenOutOfRange": true
+  }
 }

.changeset/rotten-hairs-watch.md

@@ -0,0 +1,5 @@
+---
+"@theoplayer/yospace-connector-web": minor
+---
+
+The connector now integrates with the [custom server-side ad integration API](https://www.theoplayer.com/docs/theoplayer/v7/api-reference/web/interfaces/Ads.html#registerServerSideIntegration.registerServerSideIntegration-1) introduced in THEOplayer 7.4.0. This allows Yospace adverts to show up through the `player.ads` API of THEOplayer.

cmcd/package.json

@@ -27,6 +27,7 @@
   "scripts": {
     "clean": "rimraf lib dist",
     "bundle": "rollup -c rollup.config.mjs",
+    "watch": "npm run bundle -- --watch",
     "build": "npm run clean && npm run bundle",
     "serve": "http-server",
     "test": "jest"

comscore/package.json

@@ -18,6 +18,7 @@
   "scripts": {
     "clean": "rimraf lib dist",
     "bundle": "rollup -c rollup.config.mjs",
+    "watch": "npm run bundle -- --watch",
     "build": "npm run clean && npm run bundle",
     "serve": "http-server ./.. -o /comscore/test/pages/main.html",
     "test": "echo \"No tests yet\""

conviva/package.json

@@ -19,6 +19,7 @@
   "scripts": {
     "clean": "rimraf lib dist",
     "bundle": "rollup -c rollup.config.mjs",
+    "watch": "npm run bundle -- --watch",
     "build": "npm run clean && npm run bundle",
     "serve": "http-server",
     "test": "jest"

conviva/src/utils/Utils.ts

@@ -91,12 +91,13 @@ export function collectContentMetadata(
 
 export function collectYospaceAdMetadata(player: ChromelessPlayer, ad: AdVert): ConvivaMetadata {
     return {
-        [Constants.ASSET_NAME]: ad.getProperty('AdTitle')?.getValue(),
+        // Cast to `any` because the index signature in ConvivaMetadata conflicts with its other properties...
+        [Constants.ASSET_NAME]: (ad.getProperty('AdTitle')?.getValue() ?? null) as any,
         [Constants.STREAM_URL]: player.src!,
         [Constants.DURATION]: (ad.getDuration() / 1000) as any,
         'c3.ad.technology': Constants.AdType.SERVER_SIDE,
         'c3.ad.id': ad.getIdentifier(),
-        'c3.ad.system': ad.getProperty('AdSystem')?.getValue(),
+        'c3.ad.system': (ad.getProperty('AdSystem')?.getValue() ?? null) as any,
         'c3.ad.isSlate': ad.isFiller() ? 'true' : 'false',
         'c3.ad.mediaFileApiFramework': 'NA',
         'c3.ad.adStitcher': 'YoSpace',

nielsen/package.json

@@ -18,6 +18,7 @@
   "scripts": {
     "clean": "rimraf lib dist",
     "bundle": "rollup -c rollup.config.mjs",
+    "watch": "npm run bundle -- --watch",
     "build": "npm run clean && npm run bundle",
     "serve": "http-server",
     "test": "jest"

package.json

@@ -41,7 +41,7 @@
     "prettier": "^3.2.4",
     "rimraf": "^5.0.5",
     "rollup": "^4.14.0",
-    "theoplayer": "^7.0.0",
+    "theoplayer": "^7.4.1",
     "ts-jest": "^29.1.2",
     "ts-node": "^10.9.2",
     "tslib": "^2.6.2",

yospace/package.json

@@ -28,6 +28,7 @@
     "clean": "rimraf lib dist",
     "bundle": "rollup -c rollup.config.mjs",
     "build": "npm run clean && npm run bundle",
+    "watch": "npm run bundle -- --watch",
     "serve": "http-server",
     "test": "jest"
   },