Skip to content

✨added camelCase to amp-story-shoppping component#37533

Merged
processprocess merged 4 commits intoampproject:mainfrom
jshamble:camelCaseShopping
Feb 1, 2022
Merged

✨added camelCase to amp-story-shoppping component#37533
processprocess merged 4 commits intoampproject:mainfrom
jshamble:camelCaseShopping

Conversation

@jshamble
Copy link
Copy Markdown
Contributor

@jshamble jshamble commented Feb 1, 2022

Add camelCase to amp-story-shopping component, as an accompaniment PR to the shopping config validation #37474

@amp-owners-bot amp-owners-bot bot requested a review from newmuis February 1, 2022 07:56
@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Feb 1, 2022

Hey @gmajoulet! These files were changed:

extensions/amp-story-shopping/0.1/amp-story-shopping-attachment.js
extensions/amp-story-shopping/0.1/amp-story-shopping-config.js
extensions/amp-story-shopping/0.1/amp-story-shopping-tag.js
extensions/amp-story-shopping/0.1/test/test-amp-story-shopping-attachment.js
extensions/amp-story-shopping/0.1/test/test-amp-story-shopping-config.js
extensions/amp-story-shopping/0.1/test/test-amp-story-shopping-tag.js
extensions/amp-story/1.0/amp-story-store-service.js

Hey @newmuis! These files were changed:

extensions/amp-story/1.0/amp-story-store-service.js

@processprocess
Copy link
Copy Markdown
Contributor

Awesome succinct, scoped PR. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants