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

ID5 UserId module - integrate with TrueLink Id #11802

Merged
merged 3 commits into from
Jun 19, 2024

Conversation

abazylewicz-id5
Copy link
Contributor

Type of change

  • Feature

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

ID5 offers an option for publishers to integrate a TrueLink Id on the pages. If it's present on the page, we want to capture it and pass it to the server side. We also want to expose it as a EID so its later usable as a PPID

@patmmccann
Copy link
Collaborator

Looks fine, seems like it needs docs

@abazylewicz-id5
Copy link
Contributor Author

I've added some docs about what eids are available, I can later add the same to https://docs.prebid.org/dev-docs/modules/userid-submodules/id5.html

@patmmccann
Copy link
Collaborator

patmmccann commented Jun 18, 2024

yes please make that commit, we can merge when your docs pr is opened

@patmmccann patmmccann self-requested a review June 18, 2024 18:43
@patmmccann patmmccann self-assigned this Jun 18, 2024
…tiplication

Some partners are setting refreshInSeconds as string with value '8*3600', taken from the configuration.
By using just a number we can still parse it (but we don't want to parse mathematical operations)
@patmmccann patmmccann merged commit b350eab into prebid:master Jun 19, 2024
5 checks passed
DecayConstant pushed a commit to mediavine/Prebid.js that referenced this pull request Jul 18, 2024
* ID5 User Id module - integrate with TrueLinkId

* ID5 User Id module - eids documentation

* ID5 User Id module - changed the documentation, so it doesn't use multiplication

Some partners are setting refreshInSeconds as string with value '8*3600', taken from the configuration.
By using just a number we can still parse it (but we don't want to parse mathematical operations)
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.

2 participants