Skip to content
Snippets Groups Projects
Commit bdcfbfa5 authored by Erick Hitter's avatar Erick Hitter
Browse files

Correct how JS library is loaded

parent e1f490f2
Branches
No related tags found
1 merge request!1WIP: Initial release
This commit is part of merge request !1. Comments created here will be created in the context of that merge request.
...@@ -86,7 +86,7 @@ class ETH_AMP_Analytics_WP { ...@@ -86,7 +86,7 @@ class ETH_AMP_Analytics_WP {
*/ */
public function action_wp_loaded() { public function action_wp_loaded() {
if ( ! empty( $this->get_option( 'property_id' ) ) ) { if ( ! empty( $this->get_option( 'property_id' ) ) ) {
add_filter( 'amp_component_scripts', array( $this, 'filter_amp_component_scripts' ) ); add_action( 'amp_post_template_head', array( $this, 'action_amp_post_template_head' ), 5 ); // Must appear before the AMP JS library
add_action( 'amp_post_template_footer', array( $this, 'action_amp_post_template_footer' ) ); add_action( 'amp_post_template_footer', array( $this, 'action_amp_post_template_footer' ) );
} }
} }
...@@ -98,10 +98,9 @@ class ETH_AMP_Analytics_WP { ...@@ -98,10 +98,9 @@ class ETH_AMP_Analytics_WP {
/** /**
* *
*/ */
public function filter_amp_component_scripts( $scripts ) { public function action_amp_post_template_head( $scripts ) {
$scripts['amp-analytics'] = 'https://cdn.ampproject.org/v0/amp-analytics-0.1.js'; ?><script async custom-element="amp-analytics" src="https://cdn.ampproject.org/v0/amp-analytics-0.1.js"></script>
<?php
return $scripts;
} }
/** /**
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment