Skip to content

pbs-go dev alias instructions#4087

Merged
ChrisHuie merged 1 commit intomasterfrom
pbs-aliases
Nov 17, 2022
Merged

pbs-go dev alias instructions#4087
ChrisHuie merged 1 commit intomasterfrom
pbs-aliases

Conversation

@bretg
Copy link
Contributor

@bretg bretg commented Oct 19, 2022

Added a section on how to create a hardcoded alias in PBS-Go

@bretg bretg requested a review from SyntaxNode October 19, 2022 13:11
@netlify
Copy link

netlify bot commented Oct 19, 2022

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit ad8abaa
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/634ff78627c1d70008c128cb
😎 Deploy Preview https://deploy-preview-4087--prebid-docs-preview.netlify.app/prebid-server/developers/add-new-bidder-go
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@ChrisHuie ChrisHuie added the LGTM label Oct 20, 2022
If your bidding endpoint can support more than one biddercode, you shouldn't replicate
the whole adapter codebase. Rather, follow these steps to create a 'hardcoded' alias:

1. Create a config yaml file in static/bidder-info - e.g. static/bidder-info/myalias.yaml
Copy link
Contributor

@fowler446 fowler446 Oct 21, 2022

Choose a reason for hiding this comment

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

@bretg Did you mean to have these all labeled as 1? Or should these steps be 1,2,3,4,5?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When jekyll is converted to HTML, it should create an ordered list of this syntax. Setting all lines to 1 is nice in case we want to add one in the middle later on, no need to renumber everything below.

@ChrisHuie ChrisHuie requested review from ChrisHuie and removed request for ChrisHuie October 21, 2022 15:34
@bretg bretg requested a review from fowler446 November 3, 2022 18:53
@fowler446 fowler446 removed the question label Nov 5, 2022

Notes:
- The alias name must be unique for the first 6 chars as noted above for biddercodes.
- This process will be simplified someday.
Copy link
Contributor

@fowler446 fowler446 Nov 5, 2022

Choose a reason for hiding this comment

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

lol "This process will be simplified someday."

@ChrisHuie ChrisHuie merged commit f4f5d48 into master Nov 17, 2022
@bretg bretg deleted the pbs-aliases branch July 20, 2023 15:02
ecdrsvc pushed a commit to ecdrsvc/prebid.github.io that referenced this pull request Sep 6, 2023
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