♻️ Verizon media yahoo components merge (ampproject#35797)

* Merged the VerizonMedia ad component into the yahoo component * Fixed documentation formatting issues.
BroadBandyDev · Sep 9, 2021 · a50d4c2 · a50d4c2
1 parent a4c2ae4
commit a50d4c2
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 3 deletions.
diff --git a/ads/vendors/yahoo.js b/ads/vendors/yahoo.js
@@ -5,7 +5,13 @@ import {validateData, writeScript} from '#3p/3p';
  * @param {!Object} data
  */
 export function yahoo(global, data) {
-  validateData(data, ['sid', 'site', 'sa']);
-  global.yadData = data;
-  writeScript(global, 'https://s.yimg.com/aaq/ampad/display.js');
+  if (data.sid) {
+    validateData(data, ['sid', 'site', 'sa']);
+    global.yadData = data;
+    writeScript(global, 'https://s.yimg.com/aaq/ampad/display.js');
+  } else if (data.config) {
+    validateData(data, ['config']);
+    global.jacData = data;
+    writeScript(global, 'https://jac.yahoosandbox.com/amp/jac.js');
+  }
 }
diff --git a/ads/vendors/yahoo.md b/ads/vendors/yahoo.md
@@ -14,6 +14,19 @@
 </amp-ad>
 ```
 
+```html
+<amp-ad
+  type="yahoo"
+  width="300"
+  height="250"
+  data-config='{"adServer":{"1AS":{"region":"US"}},
+    "positions":{"FB":{"alias":"1111111","sizes":["300x250"]}},
+    "site":{name:{"autoblogAMP"}},"spaceId":"111111"}'
+  data-stylesheet="https://www.autoblog.com/static/styles.css"
+>
+</amp-ad>
+```
+
 ### Configuration
 
 For configuration details, please contact https://advertising.yahoo.com/contact.
@@ -25,3 +38,27 @@ Supported parameters:
 -   `data-sid`
 -   `data-site`
 -   `data-sa`
+-   `data-config`
+-   `data-stylesheet`
+
+### Required Parameters:
+
+`data-config` - Config for ad call JAC
+`data-sa` - Config for ad call DARLA
+
+### Optional parameters:
+
+`data-stylesheet` - stylesheet to use inside iframe
+
+### Configuration Details
+
+For JAC ads : Required
+"adServer":{"1AS":{region":"US"}},
+"positions":{"FB":{alias:"1111111"},"sizes":["300x250"]}},
+"site":{name:{"autoblogAMP"}},"spaceId":"111111"}
+
+Alias, Sizes, SiteName and spaceId should be replaced by correct values.
+NOTE: SiteName should be site name + "AMP"
+
+Optional
+"params":{"name":"value"}