Skip to content

Conversation

@quietPusher
Copy link
Contributor

🏷 Type of documentation

  • new bid adapter
  • update bid adapter
  • new feature
  • text edit only (wording, typos)
  • bugfix (code examples)
  • new examples

📋 Checklist

@netlify
Copy link

netlify bot commented Mar 27, 2025

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 968b989
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/681df513232820000892cc0b
😎 Deploy Preview https://deploy-preview-5966--prebid-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See all the code


<Public title name> server-side Prebid Server adapter requires only `publisherId` and `host` parameters. But <Public title name> client-side Prebid.js adapter requires only `host`, `adUnitId`, `adUnitType`.

<Public title name> server-side Prebid Server adapter supports only `banner`, `video`, `audio`, `native` media types. But <Public title name> client-side Prebid.js adapter supports only `banner` and `video` media types, doesn't support `audio` and `native`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<Public title name> server-side Prebid Server adapter supports only `banner`, `video`, `audio`, `native` media types. But <Public title name> client-side Prebid.js adapter supports only `banner` and `video` media types, doesn't support `audio` and `native`.
The 9dotsmedialtd server-side Prebid Server adapter supports only `banner`, `video`, `audio`, `native` media types. But the 9dotsmedialtd client-side Prebid.js adapter supports only `banner` and `video` media types, doesn't support `audio` and `native`.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

remove required fields, add host example
Host to Optional
@muuki88 muuki88 added LGTM and removed needs work labels May 15, 2025

| Name | Scope | Description | Example | Type |
|:--------------|:---------|:--------------------------------------------------------------|:-----------------------|:----------|
| `publisherId` | required | Publisher ID | `'12345'` | `string` |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an alias of limelightDigital which means this also requires the host bidder param.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants