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

Dmd Id System: add rest endpoint #7066

Merged
merged 44 commits into from
Jun 24, 2021
Merged

Dmd Id System: add rest endpoint #7066

merged 44 commits into from
Jun 24, 2021

Conversation

Jayaharshak
Copy link
Contributor

Type of change

  • Feature

Description of change

Reading dmdId (dgid) from third party REST endpoint imlemented.

-contact : prebid@dmdconnects.com

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

Other information

@matthewfitz @boppudikarthikc

matthewfitz and others added 30 commits April 16, 2021 14:44
…dule

feat(module):dmd user id module[PREBID-1]
…id-params

feat(prebid):additional parameter[PREB-1]
…rage

test(prebid):added more test coverage[PREB-11]
fix(releasenote):removed unnecessary release note[PREB-11]
fix(test):updated failing test cases[PREB-11]
…rty_cookie

PREB-26 Reading third party cookies from dmdId module implemented
@ChrisHuie ChrisHuie changed the title Master Dmd Id System: add handling for Jun 22, 2021
@ChrisHuie ChrisHuie changed the title Dmd Id System: add handling for Dmd Id System: add rest endpoint Jun 22, 2021
@jsnellbaker jsnellbaker requested a review from smenzer June 22, 2021 13:53
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 bfc6f98 into prebid:master Jun 24, 2021
prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
* feat(prebid): DMD UserID Module reading from 1st party cookie [PREBID-1]

* feat(prebid):additional parameter[PREB-1]

* feat(prebid):update decode function and cacheobi[PREB-1]

* test(prebid):added more test coverage[PREB-11]

* feat(typo):cleared typo[PREB-11]

* test(prebid):updated test cases[PREB-11]

* feat(releasenote):added a release note[PREB-11]

* fix(releasenote):removed unnecessary release note[PREB-11]

* fix(test):updated failing test cases[PREB-11]

* PREB-26 Reading third party cookies from dmdId module implemented

* PREB-26 PR Comments resolved

* fix(tests):Removed ascretion make sure we can use this code in multiple envs

* fix(tests):Added ascretion to check request url

* feat(credentials): Enabling with credential flag in ajax request

* feat(credentials): Enabling with credential flag

* feat(credentials): changed callback response from object to dgid string

* ops(merge): Correct resolution on markdown conflict

* ops(merge): Correct resolution on spec conflict

* fix(conflict):Merge conflicts from upstream master are resolved

* fix(unit testing): fixed test cases

Co-authored-by: Matt Fitzgerald <matthewfitz@gmail.com>
Co-authored-by: boppudikarthikc <boppudikarthikc@gmail.com>
Co-authored-by: Karthik Boppudi <kboppudi@kboppudimac0518.local>
Co-authored-by: mfitzgerald_dmd <mfitzgerald@dmdconnects.com>
Co-authored-by: karthik <60045218+boppudikarthikc@users.noreply.github.com>
Co-authored-by: Jay Kandimalla <jkandimalla@jkandimac0421.local>
Co-authored-by: Jay kandimalla <jkandimalla@dmdconnects.com>
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.

None yet

6 participants