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

Update adikteev adapter (main PR now merged) #1066

Merged
merged 2 commits into from
Mar 26, 2019

Conversation

piotr-yuxuan
Copy link
Contributor

No description provided.

dev-docs/bidders/adikteev.md Outdated Show resolved Hide resolved
@piotr-yuxuan
Copy link
Contributor Author

@bretg should be better now.

@piotr-yuxuan
Copy link
Contributor Author

piotr-yuxuan commented Feb 15, 2019

Hello @bretg and @mike-chowla,

It looks like the adapter PR on the main repo (link) has been burried deep down the last-in-first-out PR wall 🙂 It's quite understandable because a lot of activity has happened since this adapter PR has been open two months ago. I'm trying to get this PR back to your attention, sorry for pinging you here. Would you like to review it again ?

Static analysis and CI tests have passed, and I've been checking Adikteev integration test page lately: it responds a correct bid from a test campaign. As a result, to the best of my knowledge, review may be easy.

Have a nice week-end.

@piotr-yuxuan piotr-yuxuan changed the title Update adikteev adapter Update adikteev adapter (updated) Feb 15, 2019
@piotr-yuxuan
Copy link
Contributor Author

Now the linked PR has been merged.

@piotr-yuxuan piotr-yuxuan changed the title Update adikteev adapter (updated) Update adikteev adapter (main PR merged) Mar 24, 2019
@piotr-yuxuan piotr-yuxuan changed the title Update adikteev adapter (main PR merged) Update adikteev adapter (main PR now merged) Mar 24, 2019
@bretg
Copy link
Contributor

bretg commented Mar 26, 2019

deployed with 2.8

@bretg bretg merged commit fbd2d83 into prebid:master Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants