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

✨ Google Ad Manager Publisher first-party IDs #40249

Merged
merged 17 commits into from
Mar 24, 2025

Instead of carving out proxy origin completely, just don't set the do…

d110c89
Select commit
Loading
Failed to load commit list.
Merged

✨ Google Ad Manager Publisher first-party IDs #40249

Instead of carving out proxy origin completely, just don't set the do…
d110c89
Select commit
Loading
Failed to load commit list.
amp-owners-bot / ampproject/owners-check required action Mar 21, 2025 in 8d 0h 16m 15s

Missing required OWNERS approvals!

Missing required OWNERS approvals!

Details

Current Coverage

  • [NEEDS APPROVAL] ads/google/a4a/cookie-utils.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] ads/google/a4a/test/test-cookie-utils.js (requested: erwinmombay, ychsieh)
  • build-system/test-configs/dep-check-config.js (glevitzky)
  • [NEEDS APPROVAL] extensions/amp-a4a/0.1/amp-a4a.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] extensions/amp-a4a/0.1/test/test-amp-a4a-cookies.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] extensions/amp-ad-network-adsense-impl/0.1/amp-ad-network-adsense-impl.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] extensions/amp-ad-network-adsense-impl/0.1/test/test-amp-ad-network-adsense-impl.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] extensions/amp-ad-network-doubleclick-impl/0.1/amp-ad-network-doubleclick-impl.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] extensions/amp-ad-network-doubleclick-impl/0.1/test/test-amp-ad-network-doubleclick-impl.js (requested: erwinmombay, ychsieh)
  • [NEEDS APPROVAL] src/cookies.js (requested: erwinmombay, ychsieh)
  • test/unit/test-cookies.js (glevitzky)

For a description of the OWNERS file syntax, see this example file.
To see the current ownership hierarchy and groups, see the full OWNERS tree.