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

New Adapter: Admatic #3608

Merged
merged 0 commits into from
May 2, 2024
Merged

New Adapter: Admatic #3608

merged 0 commits into from
May 2, 2024

Conversation

bakicam
Copy link
Contributor

@bakicam bakicam commented Apr 4, 2024

We added s2s support for Admatic.

Docs PR: prebid/prebid.github.io#5232

Copy link

github-actions bot commented Apr 4, 2024

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, aaafa1f

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:35:	MakeRequests		68.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:76:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:95:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:111:	MakeBids		85.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:142:	getMediaTypeForBid	88.9%
total:									(statements)		77.4%

@muuki88
Copy link

muuki88 commented Apr 4, 2024

Docs: prebid/prebid.github.io#5232

static/bidder-info/admatic.yaml Outdated Show resolved Hide resolved
static/bidder-info/admatic.yaml Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Apr 5, 2024

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 1798069

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:35:	MakeRequests		68.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:76:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:95:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:111:	MakeBids		85.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:142:	getMediaTypeForBid	88.9%
total:									(statements)		77.4%

@prebid prebid deleted a comment from github-actions bot Apr 5, 2024
@prebid prebid deleted a comment from github-actions bot Apr 5, 2024
@onkarvhanumante
Copy link
Contributor

@bakicam

consider syning Admatic-Sofware:master branch with prebid:master to resolve PR merge conflicts

@onkarvhanumante
Copy link
Contributor

onkarvhanumante commented Apr 5, 2024

@bakicam should update json test

  • to have request with multiple imps
  • to have multi-format imp request
  • request with host set as imp ext param

Copy link

github-actions bot commented Apr 5, 2024

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 1798069

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:35:	MakeRequests		68.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:76:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:95:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:111:	MakeBids		85.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:142:	getMediaTypeForBid	88.9%
total:									(statements)		77.4%

@onkarvhanumante
Copy link
Contributor

@bakicam should create and link PR for bidder doc in https:/prebid/prebid.github.io repository

@bakicam
Copy link
Contributor Author

bakicam commented Apr 9, 2024

#3608 (comment)

Hello,
PR is here prebid/prebid.github.io#5232

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 2b85726

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:35:	MakeRequests		70.6%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:73:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:92:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:108:	MakeBids		86.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:141:	getMediaTypeForBid	88.9%
total:									(statements)		78.7%

@onkarvhanumante
Copy link
Contributor

@bakicam PR has some merge conflicts. Requesting to resolve them.

@farukcam
Copy link
Contributor

#3608 (comment) Hello, we have made arrangements for this.

@farukcam
Copy link
Contributor

Hello,
Is there any progress regarding our request?

@onkarvhanumante
Copy link
Contributor

Hello, Is there any progress regarding our request?

@bakicam requesting to push updates to resolve merge conflicts

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, f5014c6

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:35:	MakeRequests		70.6%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:73:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:92:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:108:	MakeBids		86.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:141:	getMediaTypeForBid	88.9%
total:									(statements)		78.7%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 821a9ff

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:35:	MakeRequests		70.6%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:73:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:92:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:108:	MakeBids		86.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:141:	getMediaTypeForBid	88.9%
total:									(statements)		78.7%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 4544995

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:34:	MakeRequests		70.6%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:72:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:91:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:107:	MakeBids		86.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:140:	getMediaTypeForBid	88.9%
total:									(statements)		78.7%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 4ba7063

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:34:	MakeRequests		70.6%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:71:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:90:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:106:	MakeBids		86.7%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:139:	getMediaTypeForBid	88.9%
total:									(statements)		78.7%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 1099844

admatic

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:22:	Builder			80.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:34:	MakeRequests		70.6%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:71:	getImpressionExt	71.4%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:90:	resolveUrl		75.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:106:	MakeBids		100.0%
github.com/prebid/prebid-server/v2/adapters/admatic/admatic.go:132:	getMediaTypeForBid	88.9%
total:									(statements)		81.0%

@bakicam
Copy link
Contributor Author

bakicam commented Apr 25, 2024

Hello, Is there any progress regarding our request?

@bakicam requesting to push updates to resolve merge conflicts

Hello,
We are currently just waiting for approval of SyntaxNode solutions. I would like you to check it.

@farukcam
Copy link
Contributor

farukcam commented Apr 29, 2024

Hello,
We ask for your support

@onkarvhanumante
Copy link
Contributor

@bakicam @farukcam PR has some merge conflicts. PTAL to resolve them. Requesting to sync Admatic-Sofware:master with prebid:master changes

@farukcam
Copy link
Contributor

farukcam commented May 2, 2024

@onkarvhanumante I synced

@Sonali-More-Xandr Sonali-More-Xandr merged commit bc89164 into prebid:master May 2, 2024
3 checks passed
@farukcam
Copy link
Contributor

farukcam commented May 2, 2024

@onkarvhanumante @Sonali-More-Xandr Hello, I think there is a problem here. It has been merged but we cannot see the adapter.

@onkarvhanumante
Copy link
Contributor

image image

@farukcam changes force-pushed resulted into discarding file changes.

@farukcam
Copy link
Contributor

farukcam commented May 2, 2024

@onkarvhanumante I'm so sorry, I think we made a mistake. Should we create a new job request?

@onkarvhanumante
Copy link
Contributor

@onkarvhanumante I'm so sorry, I think we made a mistake. Should we create a new job request?

yes please create new PR

@farukcam
Copy link
Contributor

farukcam commented May 2, 2024

@onkarvhanumante I request you to close this prebid/prebid-server-java#3165

@onkarvhanumante
Copy link
Contributor

@onkarvhanumante I request you to close this prebid/prebid-server-java#3165

I don't have privledge to close PR from prebid-server-java repo. Added this prebid/prebid-server-java#3165 (comment) for closing PR.

@bakicam bakicam mentioned this pull request May 2, 2024
@farukcam
Copy link
Contributor

farukcam commented May 2, 2024

@onkarvhanumante Hello, We created a new application here #3654. Thank you for your support.

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.

7 participants