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

Rubicon - How to support multiple zoneIDs for one ad unit #782

Closed
aneuway2 opened this issue Nov 8, 2016 · 2 comments
Closed

Rubicon - How to support multiple zoneIDs for one ad unit #782

aneuway2 opened this issue Nov 8, 2016 · 2 comments
Assignees

Comments

@aneuway2
Copy link
Contributor

aneuway2 commented Nov 8, 2016

Type of issue

Documentation / Expected Setup

Description

There are multiple Rubicon zoneIds for one ad unit that we are trying to implement. Should we make additional calls to Rubicon within the same prebid call or whats the best way to put this in place?

The ad units accept multiple sizes on multiple templates, and the zones are tied to templates (home, channel front, article).

So first example: the rightrail_pos1 ad unit, the zoneID values are: 123, 456, 789

Steps to reproduce

Within the bidder information; multiple zoneIDs are provided for one ad unit

Expected results

No errors shown in console from Rubicon

Actual results

Seeing errors in console from Rubicon when specifying zone ids with a comma like the following:
"zoneID": "123, 456, 789"

Platform details

Prebid 0.13

@aneuway2
Copy link
Contributor Author

aneuway2 commented Nov 8, 2016

@bretg ^

@bretg
Copy link
Collaborator

bretg commented Nov 8, 2016

You can do this with the new 'rubiconLite' adapter by specifying multiple bids in the AdUnit. e.g.

var adUnits = [{
        code: 'div-gpt-ad-1460505748561-0',
        sizes: [[300, 250], [300,600]],
        bids: [{
            bidder: 'rubicon',
            params: {
               [first set of accountId/siteId/zoneId params]
            }
        },
        {
            bidder: 'rubicon',
            params: {
               [second set of accountId/siteId/zoneId params]
            }
        }]
    }

Note that we will soon be obsoleting the current 'rubicon' adapter and replacing it with the code currently in 'rubiconLite'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants