-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Does JS-injected <link rel=amphtml> trigger AMP document discovery? #6080
Comments
This depends entirely on the crawler. For some the answer will be that it On Tue, Nov 8, 2016 at 9:09 AM, Michael Stillwell notifications@github.com
|
Should we clarify this? |
Not holding @cramforce to this but I just discovered a previous related comment: "Lets just decide that JS injection is not OK." (Nov 2015) (And yes, I think some sort of clarification (even if it's "crawler-dependent but might work") would be useful.) |
@ithinkihaveacat Could you send a PR? |
@ithinkihaveacat I think I can close this with the PR merged, but please reopen if there's outstanding work. Thanks! |
https://www.ampproject.org/docs/reference/spec#amp-document-discovery explains that AMP documents are linked with their associated canonicals via the
<link rel="amphtml">
tag.Can JS be used to dynamically inject this code? i.e. via something like:
(I assume the answer is no, but it might be worthwhile making it clear in the docs.)
The text was updated successfully, but these errors were encountered: