Skip to content

Module EightPod Adapter: update bid and analytics adapters#15253

Open
ad8pod wants to merge 3 commits into
prebid:masterfrom
ad8pod:update-eightpod-adapters
Open

Module EightPod Adapter: update bid and analytics adapters#15253
ad8pod wants to merge 3 commits into
prebid:masterfrom
ad8pod:update-eightpod-adapters

Conversation

@ad8pod

@ad8pod ad8pod commented Jun 26, 2026

Copy link
Copy Markdown

Module EightPod Adapter: update bid and analytics adapters

Type of change

  • Updated bidder adapter
  • Code style update (formatting, local variables)

Description of change

This PR updates the EightPod bidder and analytics modules and normalizes the module filenames to lowercase eightpod* naming.

Changes include:

  • Updates the EightPod bid adapter to send OpenRTB banner requests with support for placement IDs, PMP deal IDs, privacy consent fields, first-party data preservation, and user.ext.eids.
  • Updates the EightPod analytics adapter to capture EightPod bidWon context and forward supported creative iframe events to Tealium CDP.
  • Expands unit coverage for bid request generation, response interpretation, privacy consent handling, EID preservation, and analytics message handling.

Other information

Documentation PR on prebid.github.io

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f140ac1da5

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread modules/eightpodBidAdapter.js
@ad8pod
ad8pod force-pushed the update-eightpod-adapters branch from f140ac1 to f2aa860 Compare June 29, 2026 08:34
@github-actions

Copy link
Copy Markdown

Whoa there partner! This project is migrating to typescript. Consider changing the new JS files to TS, with well-defined types for what interacts with the prebid public API (for example: bid params and configuration). Thanks!

  • modules/eightpodAnalyticsAdapter.js
  • modules/eightpodBidAdapter.js

@barecheck

barecheck Bot commented Jun 29, 2026

Copy link
Copy Markdown

Barecheck - Code coverage report

Total: 96.59%

Your code coverage diff: 0.01% ▴

Uncovered files and lines
FileLines
modules/eightpodAnalyticsAdapter.js171, 179-181, 192, 197, 202-203, 207-209, 221, 232
modules/eightpodBidAdapter.js135, 184, 218-220, 223

@ad8pod
ad8pod force-pushed the update-eightpod-adapters branch 5 times, most recently from 6c4ccca to e4983a6 Compare July 7, 2026 06:40
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Whoa there partner! This project is migrating to typescript. Consider changing the new JS files to TS, with well-defined types for what interacts with the prebid public API (for example: bid params and configuration). Thanks!

  • modules/eightpodAnalyticsAdapter.js
  • modules/eightpodBidAdapter.js

@ad8pod
ad8pod force-pushed the update-eightpod-adapters branch 3 times, most recently from 99f8dcf to 60414c1 Compare July 13, 2026 08:09
@ad8pod
ad8pod force-pushed the update-eightpod-adapters branch from 60414c1 to 06152f5 Compare July 14, 2026 00:17
@github-actions

Copy link
Copy Markdown

Whoa there partner! This project is migrating to typescript. Consider changing the new JS files to TS, with well-defined types for what interacts with the prebid public API (for example: bid params and configuration). Thanks!

  • modules/eightpodAnalyticsAdapter.js
  • modules/eightpodBidAdapter.js

@github-actions

Copy link
Copy Markdown

This PR introduces changes that may not work on all browsers. According to Babel, the following polyfills may be needed, and they are not automatically included:

  • Changes to modules/eightpodAnalyticsAdapter.js may need:
    • es.iterator.constructor
    • es.iterator.find
    • es.iterator.for-each
    • es.iterator.map
    • es.set.difference.v2
    • es.set.intersection.v2
    • es.set.is-disjoint-from.v2
    • es.set.is-subset-of.v2
    • es.set.is-superset-of.v2
    • es.set.symmetric-difference.v2
    • es.set.union.v2
    • esnext.iterator.constructor
    • esnext.iterator.find
    • esnext.iterator.for-each
    • esnext.iterator.map
    • esnext.set.difference.v2
    • esnext.set.intersection.v2
    • esnext.set.is-disjoint-from.v2
    • esnext.set.is-subset-of.v2
    • esnext.set.is-superset-of.v2
    • esnext.set.symmetric-difference.v2
    • esnext.set.union.v2
    • web.url-search-params.delete
    • web.url-search-params.has
    • web.url-search-params.size
  • Changes to modules/eightpodBidAdapter.js may need:
    • es.iterator.constructor
    • es.iterator.filter
    • es.iterator.find
    • es.iterator.for-each
    • es.iterator.map
    • es.iterator.some
    • esnext.iterator.constructor
    • esnext.iterator.filter
    • esnext.iterator.find
    • esnext.iterator.for-each
    • esnext.iterator.map
    • esnext.iterator.some
  • Changes to test/spec/modules/eightpodAnalyticsAdapter_spec.js may need:
    • es.iterator.constructor
    • es.iterator.filter
    • esnext.iterator.constructor
    • esnext.iterator.filter
    • web.dom-exception.stack
  • Changes to test/spec/modules/eightpodBidAdapter_spec.js may need:
    • es.array.with
    • esnext.array.with

The best way to address this is to provide good test coverage, as normal PR checks run unit tests on older browsers.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant