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

Tappx :: avoid host #3520

Merged
merged 2 commits into from
Feb 2, 2022
Merged

Tappx :: avoid host #3520

merged 2 commits into from
Feb 2, 2022

Conversation

prebidtappx
Copy link
Contributor

This changes is in response to the pull request: prebid/prebid-server#2130

This changes is in response to the pull request: prebid/prebid-server#2130
Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

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

appreciate the doc update @prebidtappx , but since you support both Prebid Server and Prebid.js, please confirm that the host param is optional for both. I don't see any PRs for PBJS.

@bretg bretg added the question label Jan 24, 2022
Change thanks to Bretg in the pull request: prebid/prebid-server#2130
I take a look to prebid.js module of tappx and we will made the same change in a near future. However, it is better to put correctly in the docs.
Thanks.
@prebidtappx
Copy link
Contributor Author

appreciate the doc update @prebidtappx , but since you support both Prebid Server and Prebid.js, please confirm that the host param is optional for both. I don't see any PRs for PBJS.

Hi! True, I made a new commit to be sure that the documentation is exactly like actual situation.

@prebidtappx
Copy link
Contributor Author

Hi bretg! The pull request was merged, we can merge this doc too?

@MartianTribe MartianTribe merged commit 176a397 into prebid:master Feb 2, 2022
jlaso pushed a commit to AuDigent/prebid.github.io that referenced this pull request Nov 6, 2024
* Tappx :: avoid host

This changes is in response to the pull request: prebid/prebid-server#2130

* Change to be more accurate

Change thanks to Bretg in the pull request: prebid/prebid-server#2130
I take a look to prebid.js module of tappx and we will made the same change in a near future. However, it is better to put correctly in the docs.
Thanks.
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