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

PulsePoint Adapter: Fixing issues related to Prebid 5.0 #6857

Merged
merged 46 commits into from
May 28, 2021
Merged

PulsePoint Adapter: Fixing issues related to Prebid 5.0 #6857

merged 46 commits into from
May 28, 2021

Conversation

anand-venkatraman
Copy link
Contributor

Type of change

  • Refactoring (no functional changes, no api changes)

Description of change

This PR addresses the issues that were marked for Prebid 5.0 release, listed below.
#6650
#6512
#6465

* ET-1691: Adding pulsepoint analytics and tests for pulsepoint adapter

* ET-1691: Adding pulsepoint analytics and tests for pulsepoint adapter

* ET-1691: cleanup

* ET-1691: minor

* ET-1691: revert package.json change
@patmmccann
Copy link
Collaborator

Excellent thank you! Can you also change pulsepointBidAdapter.md and any other doc to move the openrtb parameters to the ad unit in your video examples?

@anand-venkatraman
Copy link
Contributor Author

Docs updated in prebidBidAdapter.md and prebid.org docs prebid/prebid.github.io#2993

@patmmccann
Copy link
Collaborator

It is unclear to me what is happening with circleci tests, but we'll get this merged before the next release. Thanks!

@ChrisHuie
Copy link
Collaborator

ChrisHuie commented May 28, 2021

@anand-venkatraman @patmmccann I believe a pr that we reverted on Tuesday is the issue here. Fetch and merging upstream should fix the tests.

@anand-venkatraman
Copy link
Contributor Author

I pulled latest upstream changes and tests are passing.

@patmmccann patmmccann merged commit 9c484c0 into prebid:master May 28, 2021
prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
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