Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding new vendor IncrementX #39237

Merged
merged 15 commits into from
Jul 25, 2023
2 changes: 2 additions & 0 deletions 3p/integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,7 @@ import {imedia} from '#ads/vendors/imedia';
import {imobile} from '#ads/vendors/imobile';
import {imonomy} from '#ads/vendors/imonomy';
import {improvedigital} from '#ads/vendors/improvedigital';
import {incrementx} from '#ads/vendors/incrementx';
import {inmobi} from '#ads/vendors/inmobi';
import {innity} from '#ads/vendors/innity';
import {insticator} from '#ads/vendors/insticator';
Expand Down Expand Up @@ -430,6 +431,7 @@ register('imedia', imedia);
register('imobile', imobile);
register('imonomy', imonomy);
register('improvedigital', improvedigital);
register('incrementx', incrementx);
Prebid-Vertoz marked this conversation as resolved.
Show resolved Hide resolved
register('industrybrains', industrybrains);
register('inmobi', inmobi);
register('innity', innity);
Expand Down
12 changes: 12 additions & 0 deletions 3p/vendors/incrementx.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// src/polyfills.js must be the first import.
import '#3p/polyfills';

import {register} from '#3p/3p';
import {draw3p, init} from '#3p/integration-lib';

import {incrementx} from '#ads/vendors/incrementx';

init(window);
register('incrementx', incrementx);

window.draw3p = draw3p;
5 changes: 5 additions & 0 deletions ads/_config.js
Original file line number Diff line number Diff line change
Expand Up @@ -652,6 +652,11 @@ const adConfig = jsonConfiguration({
},
'improvedigital': {},

'incrementx': {
prefetch: 'https://cdn.incrementxserv.com/ixamp.js',
renderStartImplemented: true,
},

'industrybrains': {
prefetch: 'https://web.industrybrains.com/js/ads/async/show.js',
preconnect: [
Expand Down
14 changes: 14 additions & 0 deletions ads/vendors/incrementx.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
import {validateData, writeScript} from '#3p/3p';

/**
* @param {!Window} global
* @param {!Object} data
*/
export function incrementx(global, data) {
validateData(data, ['vzid']);
global.ixParam = data;
writeScript(
global,
'https://cdn.incrementxserv.com/ixamp.js'
);
}
20 changes: 20 additions & 0 deletions ads/vendors/incrementx.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# incrementx

## Example

```html
<amp-ad
width="320"
height="50"
type="incrementx"
data-vzid="PNXW125273VB49B96">
</amp-ad>
```

## Configuration

For configuration details and to generate your tags, please contact https://www.incrementx.com/contact-us.

Supported parameters:

- `data-vzid`
7 changes: 7 additions & 0 deletions examples/amp-ad/ads.amp.esm.html
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,7 @@
<option>imedia</option>
<option>imobile</option>
<option>imonomy</option>
<option>incrementx</option>
<option>industrybrains</option>
<option>inmobi</option>
<option>innity</option>
Expand Down Expand Up @@ -1335,6 +1336,12 @@ <h2>Imedia</h2>
data-positions='[{"id":"p1", "zoneId":"seznam.novinky.ikona2"}, {"id":"p2", "zoneId":"seznam.novinky.ikona"}]'>
</amp-ad>

<h2>IncrementX</h2>
<amp-ad width="320" height="50"
type="incrementx"
data-vzid="PNXW125273VB49B96">
</amp-ad>

<h2>Index Exchange Header Tag</h2>
<amp-ad width="300" height="250"
type="ix"
Expand Down
5 changes: 5 additions & 0 deletions examples/amp-ad/ads.amp.html
Original file line number Diff line number Diff line change
Expand Up @@ -285,6 +285,7 @@
<option>imedia</option>
<option>imobile</option>
<option>imonomy</option>
<option>incrementx</option>
<option>industrybrains</option>
<option>inmobi</option>
<option>innity</option>
Expand Down Expand Up @@ -1192,6 +1193,10 @@ <h2>Imedia</h2>
data-positions='[{"id":"p1", "zoneId":"seznam.novinky.ikona2"}, {"id":"p2", "zoneId":"seznam.novinky.ikona"}]'>
</amp-ad>

<h2>IncrementX</h2>
<amp-ad width="300" height="250" type="incrementx" data-vzid="PNXW125273VB49B96">
</amp-ad>

<h2>Index Exchange Header Tag</h2>
<amp-ad width="300" height="250" type="ix" data-ix-id="1" data-slot="/62650033/AMP_Example_Ad_Unit">
</amp-ad>
Expand Down
1 change: 1 addition & 0 deletions extensions/amp-ad/amp-ad.md
Original file line number Diff line number Diff line change
Expand Up @@ -365,6 +365,7 @@ See [amp-ad rules](validator-amp-ad.protoascii) in the AMP validator specificati
- [Imonomy](../../ads/vendors/imonomy.md)
- [Imedia](../../ads/vendors/imedia.md)
- [Improve Digital](../../ads/vendors/improvedigital.md)
- [IncrementX](../../ads/vendors/incrementx.md)
- [Insticator](../../ads/vendors/insticator.md)
- [InsurAds](../../ads/vendors/insurads.md)
- [Index Exchange](../../ads/vendors/ix.md)
Expand Down
5 changes: 5 additions & 0 deletions validator/testdata/feature_tests/ads.html
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,7 @@
<option>ibillboard</option>
<option>imobile</option>
<option>imedia</option>
<option>incrementx</option>
<option>industrybrains</option>
<option>inmobi</option>
<option>ix</option>
Expand Down Expand Up @@ -804,6 +805,10 @@ <h2>Imedia</h2>
data-positions='[{"id":"p1", "zoneId":"seznam.novinky.ikona2"}, {"id":"p2", "zoneId":"seznam.novinky.ikona"}]'>
</amp-ad>

<h2>IncrementX</h2>
<amp-ad width="320" height="50" type="incrementx" data-vzid="PNXW125273VB49B96">
</amp-ad>

<h2>Industrybrains</h2>
<amp-ad width="300" height="250"
type="industrybrains"
Expand Down
7 changes: 7 additions & 0 deletions validator/testdata/feature_tests/ads.out
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,7 @@ PASS
| <option>ibillboard</option>
| <option>imobile</option>
| <option>imedia</option>
| <option>incrementx</option>
| <option>industrybrains</option>
| <option>inmobi</option>
| <option>ix</option>
Expand Down Expand Up @@ -805,6 +806,12 @@ PASS
| data-positions='[{"id":"p1", "zoneId":"seznam.novinky.ikona2"}, {"id":"p2", "zoneId":"seznam.novinky.ikona"}]'>
| </amp-ad>
|
| <h2>IncrementX</h2>
| <amp-ad width="320" height="50"
| type="incrementx"
| data-vzid="PNXW125273VB49B96"
| </amp-ad>
|
| <h2>Industrybrains</h2>
| <amp-ad width="300" height="250"
| type="industrybrains"
Expand Down