-
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
<amp-script> must not be empty #24265
Comments
This is a developer affordance we should definitely have to set them on the correct path. Marking it as a P1 as such. |
I think this is actually due to the Is there a use case you have in mind for a child-less |
I've actually run into this as well and was confused. I was attempting to populate the Forcing a nonzero width is fine, but I'd at least expect a dev warning. |
We could do that by overriding |
What's the issue?
If
<amp-script>
is empty (doesn't contain any initial content), then no attempt is made to load it and no error is throws.How do we reproduce the issue?
This should either:
/anything.js
Which AMP version is affected?
1908222134250
The text was updated successfully, but these errors were encountered: