Skip to content

♻️ Migrate gallery 35..39 to controls#37727

Merged
kvchari merged 1 commit intoampproject:mainfrom
kvchari:migrate-gallery-to-controls
Feb 24, 2022
Merged

♻️ Migrate gallery 35..39 to controls#37727
kvchari merged 1 commit intoampproject:mainfrom
kvchari:migrate-gallery-to-controls

Conversation

@kvchari
Copy link
Copy Markdown
Contributor

@kvchari kvchari commented Feb 17, 2022

Partial: #35923

@kvchari kvchari force-pushed the migrate-gallery-to-controls branch 3 times, most recently from db2e43b to 2728b9c Compare February 17, 2022 22:46
@kvchari kvchari marked this pull request as ready for review February 17, 2022 22:46
@amp-owners-bot amp-owners-bot bot requested a review from rsimha February 17, 2022 22:46
@kvchari kvchari changed the title Migrate gallery to controls ♻️ Migrate gallery storybooks to controls Feb 17, 2022
@kvchari kvchari force-pushed the migrate-gallery-to-controls branch from 2728b9c to 8b7763b Compare February 17, 2022 22:48
@kvchari kvchari mentioned this pull request Feb 17, 2022
13 tasks
@kvchari kvchari changed the title ♻️ Migrate gallery storybooks to controls ♻️ Migrate gallery 35..39 to controls Feb 17, 2022
@kvchari kvchari enabled auto-merge (squash) February 17, 2022 22:56
@kvchari kvchari requested a review from rbeckthomas February 17, 2022 22:56
Copy link
Copy Markdown
Member

@alanorozco alanorozco left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@kvchari kvchari self-assigned this Feb 18, 2022
@kvchari kvchari linked an issue Feb 18, 2022 that may be closed by this pull request
13 tasks
'extensions/amp-soundcloud/1.0/storybook/Basic.amp.js',
'extensions/amp-soundcloud/1.0/storybook/Basic.js',
'extensions/amp-stream-gallery/1.0/storybook/Basic.amp.js',
'extensions/amp-stream-gallery/1.0/storybook/Basic.js',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should be able to approve this file but I messed up the ownership. Please pull after merging #37751, and we should be able to merge this PR.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I rebased and force pushed, but I'm still seeing a "missing owners approval" red flag

@kvchari kvchari force-pushed the migrate-gallery-to-controls branch from 8b7763b to 3470fa2 Compare February 22, 2022 21:50
@kvchari kvchari force-pushed the migrate-gallery-to-controls branch from 3470fa2 to 53dfb89 Compare February 24, 2022 17:59
@kvchari kvchari merged commit 265e0b0 into ampproject:main Feb 24, 2022
@kvchari kvchari deleted the migrate-gallery-to-controls branch February 24, 2022 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Migrate Storybook Knobs to Controls

4 participants