-
Notifications
You must be signed in to change notification settings - Fork 87
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
add userr id to bid request #880
Conversation
added unit test, also changed comment text |
Resolved. Please take a look @YuriyVelichkoPI @OlenaPostindustria . By the way, what's the common practice to contribute back? Currently this PR is created on my personal fork, and I don't have access to main repo. |
PrebidMobileTests/RenderingTests/Tests/PBMORTBAbstractTest.swift
Outdated
Show resolved
Hide resolved
PrebidMobileTests/RenderingTests/Tests/ParameterBuilderTests/ParameterBuilderServiceTest.swift
Outdated
Show resolved
Hide resolved
Hi @huanzhiNB ! Yes, it's an expected way to contribute to all prebid repositories. We plan to make a release soon. So it will be great if you finalize the PR by fixing the tests and we merge the changes. |
Hi @huanzhiNB ! Any updates about finalization? |
closes #881 |
@YuriyVelichkoPI @OlenaPostindustria Sry, had other projects going on and nearly forgot about this. I just changed the unit test case as suggested. |
PrebidMobileTests/RenderingTests/Tests/ParameterBuilderTests/ParameterBuilderServiceTest.swift
Outdated
Show resolved
Hide resolved
PrebidMobileTests/RenderingTests/Tests/PBMORTBAbstractTest.swift
Outdated
Show resolved
Hide resolved
update expected string, tested myself. This time it should be good @OlenaPostindustria |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.