Enabling Site Kit plugin breaks AMP about site-kit-wp HOT 11 CLOSED

@amitmalewar Thanks for the info. I am able to recreate the same issue. I will mark it for escalation.

To Reproduce
Steps to reproduce the behavior:

1. Install the latest version of the official AMP plugin (1.2.0)
2. Installer Site Kit 1.0.0-beta.1
3. Active Reader mode AMP
4. Visit AMP version of a post or page

Screenshots
http://recordit.co/fjjVBqLKo0

System Information (please complete the following information):

• PHP Version: 7.2
• Host OS: Linux
• Plugin Version: 1.0.0-beta.1

Additional context
No errors logged in console. Same issue arises with any theme active and no plugins other than AMP and Site Kit

from site-kit-wp.

Thanks for highlighting @amitmalewar. It is being worked on at the moment. You can disable the snippet insertion for the moment, or activate Analytics via any alternative method while it's being prioritized.

from site-kit-wp.

This QA'ed and the issue is fixed. closing.

from site-kit-wp.

@amitmalewar Thanks for the context, would you be able to let me know what plugins you are using in addition to AMP and Site Kit, or is possible share your site URL? Site Kit operates in the backend, it shouldn't affect AMP endpoints so there could be conflict. But let me know so we can take a look.

from site-kit-wp.

@jamesozzie AMP gives error only when we configure Analytics feature in Google Kit.
Site URL: https://www.infophilic.com

Plugins:

• Akismet
• Jetpack
• AMP
• Classic editor
• Contact Form 7
• Imagify
• Redirection
• Infophilic
• Subscribe to Comments Reloaded
• tagDiv composer
• Thirsty Affiliates
• Vaultpress
• WP Coupons
• WP Mail SMTP
• WP Rocket
• Yoast SEO

from site-kit-wp.

I am unable to see any AMP urls at the moment. Do you have a staging site you can use to deactivate some of your other plugins to test, and with AMP active are you seeing any errors using Chrome dev tools?

from site-kit-wp.

@jamesozzie Please check now, I have re-enabled Site Kit plugin.

from site-kit-wp.

@amitmalewar It looks like it's a problem with a plugin conflicting with your AMP plugin. Your AMP URLs do not follow the standard URL pattern (there is a / at the end of each AMP URL).

Have you installed any plugin other than Site Kit recently, or changed any settings in your active plugins? If possible please deactivate (temporarily) some of the other plugins you are using to see if your AMP works as normal (starting with the ones you recently updated or installed).

from site-kit-wp.

@amitmalewar The problem seems to only realise when Analytics is connected but the snippet is not inserted via Site Kit. Can you confirm? Deactivating Analytics also results in in AMP working as normal in reader mode.

from site-kit-wp.

@jamesozzie Yes, it crashed AMP only when we enable Analytics option. Without Analytics AMP works fine.

from site-kit-wp.

I noticed while addressing the issue that a similar problem can occur with Tag Manager in AMP reader mode. Both are addressed by #79. I updated acceptance criteria accordingly.

from site-kit-wp.