Skip to content

Comments

Company rebranding of ConnectID from Verizon Media -> Yahoo#3334

Merged
bretg merged 3 commits intoprebid:masterfrom
slimkrazy:feature/yahoo-user-identity-module-docs
Oct 21, 2021
Merged

Company rebranding of ConnectID from Verizon Media -> Yahoo#3334
bretg merged 3 commits intoprebid:masterfrom
slimkrazy:feature/yahoo-user-identity-module-docs

Conversation

@slimkrazy
Copy link
Contributor

Our supply platform now supports both the verizonmedia.com and yahoo.com eid source values. We do not want any new publishers to be exposed to the old branding. I will shortly open a PR against the existing AOL bid adapter to support the yahoo.com eid, but we also have a new Yahoo bid adapter in PR now that will replace the aol bid adapter.

Copy link
Contributor

@muuki88 muuki88 left a comment

Choose a reason for hiding this comment

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

Waiting for the prebid release on the AOL bidder ( please also link like the user module ☺️ ) and the user id module PR to get merged.


{: .alert.alert-info :}
gulp build --modules=userId,verizonMediaIdSystem
gulp build --modules=userId,connectIDIdSystem
Copy link
Contributor

Choose a reason for hiding this comment

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

Do publishers have this module in production already?

If so, I would add an additional info like
"prebid 5.13 and before use x, prebid 5.14+ use y"

Copy link
Contributor

@adam-browning adam-browning left a comment

Choose a reason for hiding this comment

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

LGTM 👍

usp_supported: true
gvl_id: 25
userIds: verizonMediaId
userIds: connectIc
Copy link
Contributor

Choose a reason for hiding this comment

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

there is a typo here

@slimkrazy
Copy link
Contributor Author

slimkrazy commented Oct 21, 2021

@ncolletti / @muuki88 - These changes have been merged and included in v5.19.0 release.

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