Skip to content

OpenX: Native support #4144

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 13 commits into from
Jan 24, 2025
Merged

OpenX: Native support #4144

merged 13 commits into from
Jan 24, 2025

Conversation

Fuska1
Copy link
Contributor

@Fuska1 Fuska1 commented Jan 9, 2025

No description provided.

Fuska1 added 12 commits December 19, 2024 13:13
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 9, 2025

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, beba8f2

openx

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/openx/openx.go:35:	MakeRequests		100.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:76:	makeRequest		75.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:122:	preprocess		85.3%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:191:	MakeBids		87.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:247:	getBidVideo		100.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:265:	getMediaTypeForImp	88.9%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:281:	Builder			100.0%
total:								(statements)		87.5%

@bsardo bsardo added the adapter label Jan 9, 2025
@Fuska1
Copy link
Contributor Author

Fuska1 commented Jan 14, 2025 •
edited
Loading

Hi, I would be very grateful if there was an opportunity to receive comments on the changes I have made. Best regards. @bsardo

@github-actions GitHub Actions
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, 2751e6f

openx

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/openx/openx.go:35:	MakeRequests		100.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:76:	makeRequest		75.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:122:	preprocess		85.3%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:191:	MakeBids		87.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:247:	getBidVideo		100.0%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:265:	getMediaTypeForImp	88.9%
github.com/prebid/prebid-server/v3/adapters/openx/openx.go:281:	Builder			100.0%
total:								(statements)		87.5%

@sergseven sergseven mentioned this pull request Jan 22, 2025
Closed
@gmiedlar-ox gmiedlar-ox mentioned this pull request Jan 23, 2025
Merged
8 tasks
@bsardo bsardo merged commit 39dba15 into prebid:master Jan 24, 2025
5 checks passed
@cross-repo-issue-creation cross-repo-issue-creation bot mentioned this pull request Jan 24, 2025
Closed
@Fuska1 Fuska1 mentioned this pull request Jan 27, 2025
Closed
@gmiedlar-ox gmiedlar-ox deleted the EXCH-11021-Add-support-for-native-imps-in-golang-Prebid-server branch January 27, 2025 07:47
scr-oath pushed a commit to scr-oath/prebid-server that referenced this pull request Mar 24, 2025
@Fuska1 @gmiedlar-ox @scr-oath
OpenX: Native support (prebid#4144)
d1cfe3a 
Co-authored-by: gmiedlar-ox <gabriela.miedlar@openx.com>
shunj-nb pushed a commit to ParticleMedia/prebid-server that referenced this pull request Jun 13, 2025
@Fuska1 @gmiedlar-ox @shunj-nb
OpenX: Native support (prebid#4144)
30cd8fe 
Co-authored-by: gmiedlar-ox <gabriela.miedlar@openx.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bsardo bsardo bsardo approved these changes

@VeronikaSolovei9 VeronikaSolovei9 VeronikaSolovei9 approved these changes

Assignees

@bsardo bsardo

@VeronikaSolovei9 VeronikaSolovei9

Labels
adapter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Fuska1 @bsardo @VeronikaSolovei9 @gmiedlar-ox