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

Compass Adapter: add new adapter #3441

Merged
merged 3 commits into from
Jan 18, 2022
Merged

Conversation

CompassSSP
Copy link
Contributor

No description provided.

@CompassSSP CompassSSP closed this Dec 2, 2021
@CompassSSP CompassSSP reopened this Dec 3, 2021
@CompassSSP CompassSSP changed the title Compass Bid Adapter: add new adapter Compass Adapter: add new adapter Dec 3, 2021

### Note

For the prebid server, you only need to use one parameter, placementId or endpointId
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we reword to:

34 -  For the prebid server, you only need to use one parameter, placementId or endpointId
   +  For the prebid server you only need to use one parameter: either placementId or endpointId

Copy link
Contributor Author

Choose a reason for hiding this comment

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

reworded

@CompassSSP
Copy link
Contributor Author

@guscarreon Our adapter has already been merged and it got into the release https://github.com/prebid/Prebid.js/releases/tag/6.4.0. Please review the doc

@CompassSSP
Copy link
Contributor Author

@mkendall07 Our adapter has already been merged and it got into the release https://github.com/prebid/Prebid.js/releases/tag/6.4.0. Please review the doc

@CompassSSP
Copy link
Contributor Author

@guscarreon @MartianTribe @ChrisHuie please react to this pull request. Prebid.js and PrebidServer adapters are already merged and added to the release, but there is no documentation

@CompassSSP
Copy link
Contributor Author

@guscarreon @MartianTribe @ChrisHuie @fowler446 @bretg please someone answer

@mkendall07 mkendall07 merged commit b2f2fac into prebid:master Jan 18, 2022
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