Skip to content

Conversation

@bretg
Copy link
Contributor

@bretg bretg commented Mar 4, 2025

Adding notes for how to update the codepath notifications for the various repos.

@netlify
Copy link

netlify bot commented Mar 4, 2025

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit f78322a
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/67c774a9e202b600082a5cf9
😎 Deploy Preview https://deploy-preview-5922--prebid-docs-preview.netlify.app
📱 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 requested a review from muuki88 March 4, 2025 21:51
Copy link
Contributor

@muuki88 muuki88 left a comment

Choose a reason for hiding this comment

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

Wording is good. What do you think making this text an include , so the text doesn't need to be copy pasted. Maybe this will be added for mobile in the future as well

@bretg
Copy link
Contributor Author

bretg commented Mar 6, 2025

Thanks Muki, but the links in each are different in each. I think there is a way to pass arguments into an include, but is it really worth it for this? I don't see mobile getting this anytime soon because there aren't any vendor-specific modules.

@bretg bretg merged commit c6dc9ad into master Mar 31, 2025
6 checks passed
@bretg bretg deleted the codepath-docs branch March 31, 2025 15:25
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.

3 participants