Skip to content

adomik Analytics Adapter: sampling revamp, add AB test info, enhance bidWon info#8122

Merged
ChrisHuie merged 5 commits intoprebid:masterfrom
liabas-b:adomik-adapter-save-won-bidresp
Apr 20, 2022
Merged

adomik Analytics Adapter: sampling revamp, add AB test info, enhance bidWon info#8122
ChrisHuie merged 5 commits intoprebid:masterfrom
liabas-b:adomik-adapter-save-won-bidresp

Conversation

@liabas-b
Copy link
Contributor

@liabas-b liabas-b commented Feb 25, 2022

Type of change

  • Bugfix
  • Feature
  • New bidder adapter
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Does this change affect user-facing APIs or examples documented on http://prebid.org?
  • Other

Description of change

  1. Forward Adomik A/B test information in AdomikAnalyticsAdapter
  2. Enhance and refactor sampling
  3. Add information for recorded bidWon events

@liabas-b liabas-b changed the title Adomik adapter save won bidresp adomikAnalyticsAdapter: sampling revamp, add AB test info, enhance bidWon info Feb 25, 2022
@robertrmartinez robertrmartinez requested a review from lksharma March 1, 2022 18:42
@robertrmartinez
Copy link
Collaborator

@liabas-b Please add associated unit tests as well!

@liabas-b
Copy link
Contributor Author

liabas-b commented Mar 2, 2022

@liabas-b Please add associated unit tests as well!

Work in progress :)

@liabas-b liabas-b force-pushed the adomik-adapter-save-won-bidresp branch from e61ec14 to 8e05d27 Compare March 3, 2022 10:57
@liabas-b
Copy link
Contributor Author

liabas-b commented Mar 3, 2022

@liabas-b Please add associated unit tests as well!

Done @robertrmartinez and @lksharma. Any advice will be appreciated I'm no javascript specialist :)

@patmmccann patmmccann changed the title adomikAnalyticsAdapter: sampling revamp, add AB test info, enhance bidWon info adomik Analytics Adapter: sampling revamp, add AB test info, enhance bidWon info Mar 9, 2022
@liabas-b
Copy link
Contributor Author

@patmmccann @lksharma Hello, is anything still blocking the merge ?
Thanks, Benoit.

1 similar comment
@liabas-b
Copy link
Contributor Author

liabas-b commented Apr 4, 2022

@patmmccann @lksharma Hello, is anything still blocking the merge ?
Thanks, Benoit.

@patmmccann
Copy link
Collaborator

just review. @ChrisHuie ; could you investigate if this reviewer is on leave?

@ChrisHuie ChrisHuie self-requested a review April 13, 2022 16:59
@ChrisHuie ChrisHuie self-assigned this Apr 13, 2022
@ChrisHuie ChrisHuie merged commit b56b3bc into prebid:master Apr 20, 2022
JoelPM pushed a commit to JoelPM/Prebid.js that referenced this pull request Apr 25, 2022
…bidWon info (prebid#8122)

* Adomik Analytics Adapter: Save bid response of the won event

* wip

* adomikAnayticsAdapter/receive KV from sessionStorage

* AdomikAnalyticsAdapter: add specs

* kick off testing

Co-authored-by: thomasferal <thomas.feral@adomik.com>
Co-authored-by: Chris Huie <phoenixtechnerd@gmail.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.

6 participants

Comments