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

Eplanning Bid Adapter: viewability refactor and fix in search the divs #8610

Merged
merged 4 commits into from
Jul 12, 2022
Merged

Eplanning Bid Adapter: viewability refactor and fix in search the divs #8610

merged 4 commits into from
Jul 12, 2022

Conversation

fndigrazia
Copy link
Contributor

Type of change

  • [ x] Bugfix
  • [ x] Refactoring (no functional changes, no api changes)

Description of change

eplanning viewability refactor and fix in search the divs

[email protected]

  • [ x] official adapter submission

@ChrisHuie ChrisHuie changed the title eplanning viewability refactor and fix in search the divs Eplanning Bid Adapter: viewability refactor and fix in search the divs Jun 28, 2022
@ChrisHuie ChrisHuie requested a review from Rothalack June 28, 2022 13:44
@sebaperez
Copy link
Contributor

Hi @Rothalack , do you have any ETA for reviewing this? Thanks!

@Rothalack
Copy link
Collaborator

Sorry about that, been crazy busy.

This looks good and is working in testing. I'm confused why it didn't get a CircleCI build, I've never seen that before. Do you have an idea why it doesn't have a cci build?

@Rothalack
Copy link
Collaborator

I'm not seeing any other PR's that have done this. My best guess is maybe try to do a master merge and commit that? A commit may trigger the build.

@sebaperez
Copy link
Contributor

Sure, thanks! We just did a new commit so it should trigger the build

@Rothalack
Copy link
Collaborator

I'm not sure why that's not triggering a build. I've reached out on slack to see if I can get it figured out.

@ChrisHuie
Copy link
Collaborator

@Rothalack

I'm not sure why that's not triggering a build. I've reached out on slack to see if I can get it figured out.

Kicked off tests 👍 (gave a bit more detail on this issue over slack)

Copy link
Collaborator

@Rothalack Rothalack left a comment

Choose a reason for hiding this comment

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

Good to go!

@Rothalack Rothalack merged commit cdbc08a into prebid:master Jul 12, 2022
bwhisp pushed a commit to bwhisp/Prebid.js that referenced this pull request Jul 13, 2022
prebid#8610)

* eplanning viewability refactor and fix in search the divs

* change observer configuration name

* unused variable is removed

* kick off circleci tests manually

Co-authored-by: Chris Huie <[email protected]>
ahmadlob referenced this pull request in taboola/Prebid.js Jul 27, 2022
…s (#8610)

* eplanning viewability refactor and fix in search the divs

* change observer configuration name

* unused variable is removed

* kick off circleci tests manually

Co-authored-by: Chris Huie <[email protected]>
RomainLofaso pushed a commit to criteo-forks/Prebid.js that referenced this pull request Aug 8, 2022
prebid#8610)

* eplanning viewability refactor and fix in search the divs

* change observer configuration name

* unused variable is removed

* kick off circleci tests manually

Co-authored-by: Chris Huie <[email protected]>
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