Skip to content

Comments

PBS us custom logic module docs#4883

Merged
bretg merged 5 commits intomasterfrom
pbs-uscustomlogic
Sep 28, 2023
Merged

PBS us custom logic module docs#4883
bretg merged 5 commits intomasterfrom
pbs-uscustomlogic

Conversation

@bretg
Copy link
Contributor

@bretg bretg commented Sep 22, 2023

No description provided.

@netlify
Copy link

netlify bot commented Sep 22, 2023

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit bcfbc94
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/6515b9b7c438550008a50178
😎 Deploy Preview https://deploy-preview-4883--prebid-docs-preview.netlify.app/prebid-server/features/pbs-uscustomlogic
📱 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 configuration.

@bretg bretg changed the title pbs us custom logic initial control PBS us custom logic module docs Sep 25, 2023
@muuki88
Copy link
Contributor

muuki88 commented Sep 27, 2023

@bretg this looks good to me. I can't say too much on the technical details, but I could follow everything and feel confident that if I should ever have the pleasure to implement a custom us consent logic, this would help me get started. Even though I hope this day may never come 😅

bretg and others added 2 commits September 28, 2023 13:31
Co-authored-by: Muki Seiler <muuki88@users.noreply.github.com>
converting table to list
@bretg
Copy link
Contributor Author

bretg commented Sep 28, 2023

Thanks for the review @muuki88 . Going to go ahead and merge. Converting the legal warning to an include can happen as a separate PR.

@bretg bretg merged commit cc8c8ee into master Sep 28, 2023
@bretg bretg deleted the pbs-uscustomlogic branch September 28, 2023 17:39
@muuki88
Copy link
Contributor

muuki88 commented Sep 28, 2023

Sure thing. Maybe I'll find time tomorrow for refactoring this

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.

2 participants