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

amp-analytics: Support amp-google-client-id-api lookup in AmpDocShadow #26683

Merged
merged 2 commits into from
Feb 11, 2020

Conversation

mdmower
Copy link
Contributor

@mdmower mdmower commented Feb 9, 2020

  • Read meta[name="amp-google-client-id-api"] using method supported by
    AmpDocShadow as well as other AmpDoc subtypes.
  • Tag <body> with i-amphtml-linker-created in AmpDocShadow since <meta>
    element does not exist.

- Read meta[name="i-amphtml-access-state"] using method supported by
  AmpDocShadow as well as other AmpDoc subtypes.
- Tag <body> with i-amphtml-linker-created in ShadowRoot since <meta>
  element does not exist.
@amp-owners-bot amp-owners-bot bot requested a review from micajuine-ho February 9, 2020 00:58
@mdmower mdmower changed the title amp-analytics: Support amp-google-client-id-api lookup in ShadowDoc amp-analytics: Support amp-google-client-id-api lookup in AmpDocShadow Feb 9, 2020
Query meta[name="amp-google-client-id-api"] using method supported by
AmpDocShadow as well as other AmpDoc subtypes.
Copy link
Contributor

@micajuine-ho micajuine-ho left a comment

Choose a reason for hiding this comment

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

Thanks for the improvement!

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

Successfully merging this pull request may close these issues.

4 participants