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

added adaptor for sovrn #83

Closed
wants to merge 1 commit into from
Closed

added adaptor for sovrn #83

wants to merge 1 commit into from

Conversation

vikpj
Copy link

@vikpj vikpj commented Oct 25, 2015

Hi all,

I've created an adaptor for Sovrn. Please review and let me know if you have any questions or concerns.

@vikpj vikpj force-pushed the sovrn branch 3 times, most recently from 775686b to 7cb4f1b Compare October 26, 2015 04:52
@ialex
Copy link
Contributor

ialex commented Oct 27, 2015

Is it running on a live page i can check? i would love to see the adaptor working.

@vikpj
Copy link
Author

vikpj commented Oct 28, 2015

I've been testing it here: http://exposuredynamics.com/test/prebid_test.html.

There are four caveats on the test page.

  1. Instead of sending location.hostname it sends a static string "www.healthstatus.com". This is obviously not part of the commit.
  2. I'm using the test url adintegration.lijit.com/rtb/bid instead of the production url ap.lijit.com/rtb/bid. This is obviously not part of the commit.
  3. I'm using prebid.js instead of prebid.min.js.
  4. I'm testing postbid instead of prebid.

@mkendall07
Copy link
Member

Hi @vikpj ,

Thank you for you for contributing. We are currently working directly with Sovrn for an official adapter. I expect that will be pulled into prebid.js very soon. Thanks

@vikpj
Copy link
Author

vikpj commented Nov 4, 2015

@mkendall07 I actually worked directly with Sovrn to build this.

@vikpj vikpj closed this Nov 15, 2015
@vikpj vikpj deleted the sovrn branch November 15, 2015 19:55
github-baptiste-haudegand pushed a commit to ebuzzing/Prebid.js that referenced this pull request May 21, 2024
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.

3 participants