Skip to content

update NextMillennium bid adapter doc#3336

Closed
mifanich wants to merge 2 commits intoprebid:masterfrom
nextmillenniummedia:update-nm-doc
Closed

update NextMillennium bid adapter doc#3336
mifanich wants to merge 2 commits intoprebid:masterfrom
nextmillenniummedia:update-nm-doc

Conversation

@mifanich
Copy link
Contributor

No description provided.

@SyntaxNode
Copy link
Contributor

SyntaxNode commented Oct 11, 2021

The NextMillennium adapter sends only the placementID to their bidding server. Could you please add a comment in this doc to make it clear that no other information from the request, such as bid floors, are sent to NextMillennium.

@mifanich
Copy link
Contributor Author

The NextMillennium adapter sends only the placementID to their bidding server. Could you please add a comment in this doc to make it clear that no other information from the request, such as bid floors, are sent to NextMillennium.

Thank you for comment!
I have updated the doc and sent a new commit. Also I confirm, that we accept only placement ID, us_privacy and gdpr

bretg added a commit that referenced this pull request Oct 21, 2021
Replacement for #3336, easier than fixing those conflicts
@bretg bretg mentioned this pull request Oct 21, 2021
bretg added a commit that referenced this pull request Oct 21, 2021
Replacement for #3336, easier than fixing those conflicts
@bretg bretg added LGTM and removed needs work labels Oct 21, 2021
@bretg
Copy link
Contributor

bretg commented Oct 21, 2021

There were conflicts here, which I resolved with #3364. Also move the description out of the metadata into the body.

@bretg bretg closed this Oct 21, 2021
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

Comments