Skip to content

Update amp-twitter validation for moments and timelines#23137

Merged
cathyxz merged 1 commit intoampproject:masterfrom
cathyxz:bugfix/amp-twitter
Jul 2, 2019
Merged

Update amp-twitter validation for moments and timelines#23137
cathyxz merged 1 commit intoampproject:masterfrom
cathyxz:bugfix/amp-twitter

Conversation

@cathyxz
Copy link
Copy Markdown
Contributor

@cathyxz cathyxz commented Jul 1, 2019

Closes #20340.

The attributes theme and link-color are applicable to Tweets, Moments and Timelines. Since we already validate that one of data-momentid, data-timeline-source-type, or data-tweetid must be present, there is no need to additionally validate these since we pass on all data- attributes.

@cathyxz cathyxz force-pushed the bugfix/amp-twitter branch from d0d87c2 to 8b80c22 Compare July 1, 2019 18:26
@cathyxz cathyxz marked this pull request as ready for review July 1, 2019 18:28
@cathyxz cathyxz requested review from Gregable and sparhami July 1, 2019 19:35
@cathyxz cathyxz requested review from aghassemi and cvializ and removed request for aghassemi and sparhami July 1, 2019 22:05
Copy link
Copy Markdown
Contributor

@cvializ cvializ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@cathyxz cathyxz merged commit 5809993 into ampproject:master Jul 2, 2019
@cathyxz cathyxz deleted the bugfix/amp-twitter branch July 2, 2019 19:27
twifkak added a commit to twifkak/amphtml that referenced this pull request Jul 9, 2019
@twifkak twifkak mentioned this pull request Jul 9, 2019
twifkak added a commit that referenced this pull request Jul 9, 2019
* cl/257092185 Revision bump for #23137

* cl/257092258 Revision bump for #23143

* cl/257092328 Revision bump for #22917

* cl/257092384 Revision bump for #23180

* cl/257092442 Revision bump for #23173

* cl/257092491 Revision bump for #23195
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[amp-twitter] validation error when using data-theme without data-tweetid

4 participants