Skip to content
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

Adding amp-youtube to amp-story-grid #824

Open
TheRook opened this issue Sep 26, 2023 · 0 comments
Open

Adding amp-youtube to amp-story-grid #824

TheRook opened this issue Sep 26, 2023 · 0 comments

Comments

@TheRook
Copy link

TheRook commented Sep 26, 2023

I can see that amp-youtube isn't allowed, so it must be more difficult than simply adding it to the dependents list:
https://github.com/ampproject/amphtml/blob/main/extensions/amp-story/validator-amp-story.protoascii#L749-L750

What is involved in fixing this roadblock? I feel like amp stories would benefit greatly from allowing amp-youtube, seeing as amp-twitter and amp-video is already being used there should be a way for the community to fix this issue.

What is involved in fixing this short coming of amp-stories?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant