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

Contxtful Bid Adapter : initial release #12256

Merged
merged 18 commits into from
Oct 24, 2024

Conversation

sebastienrufiange
Copy link
Contributor

@sebastienrufiange sebastienrufiange commented Sep 19, 2024

Type of change

  • New bidder adapter

Description of change

  • Adds the Contxtful Bidder Adapter

Other information

@sebastienrufiange sebastienrufiange marked this pull request as ready for review September 20, 2024 18:06
@ChrisHuie ChrisHuie changed the title Contxtful Bid Adapter: New adapter Contxtful Bid Adapter : initial release Sep 27, 2024
@sebastienrufiange
Copy link
Contributor Author

Hello @ChrisHuie and @lksharma,
Thanks in advance for the time spent to review, it is well appreciated!
I'll be reactive during this review process.
Best.

import { registerBidder } from '../src/adapters/bidderFactory.js';
import { BANNER, NATIVE, VIDEO } from '../src/mediaTypes.js';
import { _each, buildUrl, isStr, isEmptyStr, logInfo, logError } from '../src/utils.js';
import * as ajax from '../src/ajax.js';
Copy link
Collaborator

Choose a reason for hiding this comment

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

import what you need please

Copy link
Contributor Author

@sebastienrufiange sebastienrufiange Oct 24, 2024

Choose a reason for hiding this comment

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

@patmmccann Thank you for pointing this out! I fixed it in this commit

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.

tiny tweak please

@patmmccann patmmccann assigned patmmccann and unassigned lksharma Oct 24, 2024
@patmmccann patmmccann removed the request for review from lksharma October 24, 2024 15:17
@ChrisHuie ChrisHuie merged commit 51f329e into prebid:master Oct 24, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants