Skip to content

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

Merged
mdmower merged 2 commits intoampproject:masterfrom
mdmower:pr-analytics
Feb 11, 2020
Merged

amp-analytics: Support amp-google-client-id-api lookup in AmpDocShadow#26683
mdmower merged 2 commits intoampproject:masterfrom
mdmower:pr-analytics

Conversation

@mdmower
Copy link
Copy Markdown
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
Copy Markdown
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