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
Extension analytics to load immediately #8759
Conversation
Looks good! Is the |
@zhouyx Yes. Temporary. This is not a user-exposed attribute. The judgement for now goes as this: the parent element already follows system priority and, essentially, the same priority is applied to analytics. That does mean that ideally we'd not call |
@zhouyx Tests are ready. PTAL. |
/cc @keithwrightbos @tdrl |
Excellent! Thank you! So if I understand this PR, we don't need to change our code, correct? (E.g., don't need to insert a |
Correct
…On Apr 14, 2017 5:28 AM, "Terran Lane" ***@***.***> wrote:
Excellent! Thank you! So if I understand this PR, we *don't* need to
change our code, correct? (E.g., don't need to insert a trigger:
'immediate', because insertAnalyticsElement has already covered that for
us?)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#8759 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ANd2kJsEWOdNa1BUlbHSlod2-kSjR3Trks5rv2Z9gaJpZM4M9XRh>
.
--
You received this message because you are subscribed to the Google Groups
"amphtml-eng-github" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to ***@***.***
To post to this group, send email to ***@***.***
To view this discussion on the web visit https://groups.google.com/a/
google.com/d/msgid/amphtml-eng-github/ampproject/amphtml/
pull/8759/c294145689%40github.com
<https://groups.google.com/a/google.com/d/msgid/amphtml-eng-github/ampproject/amphtml/pull/8759/c294145689%40github.com?utm_medium=email&utm_source=footer>
.
--
You received this message because you are subscribed to the Google Groups
"amphtml-eng" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to ***@***.***
To post to this group, send email to ***@***.***
To view this discussion on the web visit https://groups.google.com/a/
google.com/d/msgid/amphtml-eng/ampproject/amphtml/pull/
8759/c294145689%40github.com
<https://groups.google.com/a/google.com/d/msgid/amphtml-eng/ampproject/amphtml/pull/8759/c294145689%40github.com?utm_medium=email&utm_source=footer>
.
|
* Extension analytics to load immediately * fix types * tests
* Extension analytics to load immediately * fix types * tests
Closes #8752.
TODO: