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.

Sign up for GitHub

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

Jump to bottom

📖 Update`<amp-consent>` `policy` configuration snippet #35362

Merged

alanorozco merged 2 commits into main from alanorozco-patch-19

Jul 26, 2021

Member

alanorozco commented Jul 22, 2021 •

edited

Update snippet to valid JSON
Update some wording
Unrelated: Fixes AppConsent website URL in order to pass check-links with this change.


          📖 Update<amp-consent> policy configuration snippet

1695bdf

1. Update snippet to valid JSON
2. Update some wording

alanorozco requested a review from caroqliu

July 22, 2021 19:19

caroqliu approved these changes

View reviewed changes


          Update amp-consent.md

1cc4581

alanorozco merged commit 0b34e0e into main

ampprojectbot added the PR use: In Stable label

rsimha deleted the alanorozco-patch-19 branch

August 27, 2021 16:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment