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

Refine PBS activity attributes #5106

Merged
merged 3 commits into from
Apr 17, 2024
Merged

Refine PBS activity attributes #5106

merged 3 commits into from
Apr 17, 2024

Conversation

bretg
Copy link
Contributor

@bretg bretg commented Jan 26, 2024

As noted in prebid/prebid-server#3304

Spoke with Prebid's legal counsel. She agreed that Prebid Server could reasonably preserve device.geo.country and device.geo.region.

Copy link

netlify bot commented Jan 26, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit a83a066
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/661fd7e47dd5bc0008fa8811
😎 Deploy Preview https://deploy-preview-5106--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
Copy link
Contributor Author

bretg commented Jan 26, 2024

the code doesn't do this yet, so please don't merge the docs yet

Co-authored-by: Markiyan Mykush <95693607+marki1an@users.noreply.github.com>
@bretg
Copy link
Contributor Author

bretg commented Apr 17, 2024

Thanks @marki1an - I'd forgotten about this old PR. It's got conflicts now. Will fix...

@bretg bretg removed the on hold label Apr 17, 2024
@bretg
Copy link
Contributor Author

bretg commented Apr 17, 2024

added a note about which version does this, so going ahead with the merge.

@bretg bretg merged commit e424dd9 into master Apr 17, 2024
5 checks passed
@bretg bretg deleted the bretg-patch-2 branch April 17, 2024 14:10
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.

None yet

2 participants