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

akamaiDAPIdSystem:add new ID submodule #7084

Merged
merged 1 commit into from
Jun 25, 2021
Merged

Conversation

amishra11j
Copy link
Contributor

Type of change

  • Feature

Description of change

Adding the akamaiDAPId userId submodule.

contact email of maintainer: amishra@akamai.com

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:
PR in docs repo: prebid/prebid.github.io#3058

Copy link
Collaborator

@smenzer smenzer left a comment

Choose a reason for hiding this comment

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

please add tests to the userId_spec.js file, otherwise this LGTM

apiHostname: 'api.dap.akadns.net',
domain: 'forums-jedi.akamaized.net',
type: 'email',
identity: 'aaryn@query.com'
Copy link
Collaborator

Choose a reason for hiding this comment

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

this field isn't documented anywhere...should it be?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @smenzer for highlighting this. These fields are mentioned in 'akamaiDAPIdSystem.md' readme file but I missed adding 'identity' field. I have updated it and also updated 'userId.md' file with same change.
Have provisioned a new (apiHostname: 'prebid.dap.akadns.net') for Prebid.org so updated the integration examples and spec files.
I have added tests to userId_spec.js file as well.

Copy link
Collaborator

@smenzer smenzer left a comment

Choose a reason for hiding this comment

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

LGTM

@smenzer smenzer merged commit 4c20820 into prebid:master Jun 25, 2021
@amishra11j amishra11j deleted the MPT-417 branch June 28, 2021 04:07
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.

4 participants