Skip to content

EX.CO Adapter: viewability measurement, sync, bid won and timeout pixels support #13047

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

pb-pete
Copy link
Contributor

@pb-pete pb-pete commented Apr 29, 2025

Type of change

  • Updated bidder adapter

Description of change

  • Timeout Pixel support
  • Bid Won Pixel support
  • Sync Pixels support
  • Viewability measurement support

return str.replace('[TIMESTAMP]', Date.now());
}

percentInViewport(win, element) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think you can import all this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

have not found proper util to fit our needs

Copy link
Collaborator

@patmmccann patmmccann left a comment

Choose a reason for hiding this comment

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

comments in line

@pb-pete
Copy link
Contributor Author

pb-pete commented May 5, 2025

comments in line

thanks for a review, all comments answered / handled

@pb-pete
Copy link
Contributor Author

pb-pete commented May 19, 2025

@ChrisHuie @patmmccann anything more needs to be adjusted to make this PR approved?

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.

3 participants