Skip to content

Commit

Permalink
Merge pull request #165 from renzit/master
Browse files Browse the repository at this point in the history
Add amp accordeon to component mapping
  • Loading branch information
Sigurður Guðbrandsson authored Apr 18, 2017
2 parents 160aa98 + 1decfbb commit ea60f9c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/Validate/Context.php
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ class Context
'amp-vimeo' => 'https://cdn.ampproject.org/v0/amp-vimeo-0.1.js',
'amp-youtube' => 'https://cdn.ampproject.org/v0/amp-youtube-0.1.js',
'amp-sidebar' => 'https://cdn.ampproject.org/v0/amp-sidebar-0.1.js',
'amp-accordion' => 'https://cdn.ampproject.org/v0/amp-accordion-0.1.js',
'template' => 'https://cdn.ampproject.org/v0/amp-mustache-0.1.js'
];

Expand Down
3 changes: 2 additions & 1 deletion tests/test-data/full-html/validator-amp-accordion.html.out
Original file line number Diff line number Diff line change
Expand Up @@ -166,4 +166,5 @@ FAIL

COMPONENT NAMES WITH JS PATH
------------------------------
No custom amp script includes required
'amp-accordion', include path 'https://cdn.ampproject.org/v0/amp-accordion-0.1.js'

1 change: 1 addition & 0 deletions tests/test-data/full-html/validator-amp-sidebar.html.out
Original file line number Diff line number Diff line change
Expand Up @@ -137,5 +137,6 @@ FAIL
COMPONENT NAMES WITH JS PATH
------------------------------
'amp-sidebar', include path 'https://cdn.ampproject.org/v0/amp-sidebar-0.1.js'
'amp-accordion', include path 'https://cdn.ampproject.org/v0/amp-accordion-0.1.js'
'amp-fit-text', include path 'https://cdn.ampproject.org/v0/amp-fit-text-0.1.js'

0 comments on commit ea60f9c

Please sign in to comment.