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

Updating smartadserver adapter for prebid v3 compatibility. #4714

Merged
merged 1 commit into from
Jan 21, 2020

Conversation

tadam75
Copy link
Contributor

@tadam75 tadam75 commented Jan 10, 2020

Type of change

  • Feature

Description of change

We updated our smartadserver adapter for prebid v3 compatibility by removing deprecated functions and correctly using adunit sizes.

@mlapeyre3
Copy link

Hello guys!

Can you give us a heads up on the status of our pull-request?
It seems the failing tests of the CI are due to other adapters, but the big red needs review label looks like intimidating :)

Cheers!

@mlapeyre3
Copy link

Hello @idettman, @jsnellbaker,

Any updates on this?

Thanks!

Copy link
Collaborator

@jsnellbaker jsnellbaker left a comment

Choose a reason for hiding this comment

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

LGTM

@jsnellbaker jsnellbaker merged commit 1b591f2 into prebid:master Jan 21, 2020
hellsingblack pushed a commit to SublimeSkinz/Prebid.js that referenced this pull request Mar 5, 2020
@tadam75 tadam75 deleted the prebid-3-smartadserver branch April 3, 2020 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants