Skip to content

Update amp-date-display validation to allow template attribute#23866

Merged
cathyxz merged 2 commits intoampproject:masterfrom
cathyxz:feature/date-display
Aug 12, 2019
Merged

Update amp-date-display validation to allow template attribute#23866
cathyxz merged 2 commits intoampproject:masterfrom
cathyxz:feature/date-display

Conversation

@cathyxz
Copy link
Copy Markdown
Contributor

@cathyxz cathyxz commented Aug 9, 2019

Closes #23731.

Context: amp-date-display makes use of <template>s. It should therefore allow the template attribute to reference templates without causing template nesting when used inside of <amp-list>.

@cathyxz cathyxz requested a review from Gregable August 9, 2019 18:05
@amp-bundle-size amp-bundle-size bot requested a review from dreamofabear August 9, 2019 19:15
@Gregable
Copy link
Copy Markdown
Member

Gregable commented Aug 9, 2019

fyi @twifkak as this might interact with the bug you are working on.

@cathyxz cathyxz merged commit 8a19698 into ampproject:master Aug 12, 2019
@cathyxz cathyxz deleted the feature/date-display branch August 12, 2019 17:32
Gregable pushed a commit that referenced this pull request Aug 16, 2019
* cl/262591581 Revision bump for #23840

* cl/262631527 Revision bump for #23765

* cl/263233041 data-ampdevmode. Avoid using non-data attributes on non-custom elements.

* cl/263556980 Allow nonce on `<link rel=stylesheet>`.

* cl/263618983 Revision bump for #23775

* cl/263623541 Revision bump for #23873

* cl/263628558 Revision bump for #23866

* cl/263637721 Revision bump for #23867

* cl/263656022 Revision bump for #23967

* fix reverse sync

* fix reverse sync
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-date-display inside amp-list does not render correctly

4 participants