Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(store-ui): Adding select atom #876

Merged
merged 19 commits into from
Aug 23, 2021
Merged

feat(store-ui): Adding select atom #876

merged 19 commits into from
Aug 23, 2021

Conversation

bentoper
Copy link
Contributor

@bentoper bentoper commented Aug 3, 2021

What's the purpose of this pull request?

Adds the Select atom to store-ui

@bentoper bentoper requested a review from a team as a code owner August 3, 2021 16:59
@netlify
Copy link

netlify bot commented Aug 3, 2021

✔️ Deploy Preview for storeui ready!

🔨 Explore the source changes: 9a8546a

🔍 Inspect the deploy log: https://app.netlify.com/sites/storeui/deploys/6123bd2e64163c0007024f63

😎 Browse the preview: https://deploy-preview-876--storeui.netlify.app

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 3, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9a8546a:

Sandbox Source
Store UI Typescript Configuration

Copy link
Contributor

@tlgimenes tlgimenes left a comment

Choose a reason for hiding this comment

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

Please export the atom and add it to the storybook. Also, tests are missing

@bentoper bentoper marked this pull request as draft August 4, 2021 18:14
@bentoper bentoper marked this pull request as ready for review August 5, 2021 15:46
@bentoper bentoper requested a review from tlgimenes August 5, 2021 16:08
@bentoper bentoper force-pushed the feat/select-atom branch 2 times, most recently from 6f804c6 to bbde42c Compare August 6, 2021 16:23
@bentoper bentoper dismissed tlgimenes’s stale review August 6, 2021 17:23

Resolved issues

Copy link
Contributor

@tlgimenes tlgimenes left a comment

Choose a reason for hiding this comment

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

Awesome! Everything is great! however, can you make a PR upgrading marin/btglobal/storecomponents/c4 so we know there is not breaking change on them? If there is, pleases upgrade in there too, thanks!

Copy link
Contributor

@igorbrasileiro igorbrasileiro left a comment

Choose a reason for hiding this comment

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

Nice job @bentoper ! Left some comments. Remember to update the stores with this breaking change, ok?!

packages/store-ui/src/atoms/Select/Select.tsx Outdated Show resolved Hide resolved
packages/store-ui/src/atoms/Select/Select.test.tsx Outdated Show resolved Hide resolved
packages/store-ui/src/deprecated/index.ts Show resolved Hide resolved
packages/store-ui/src/atoms/Select/Select.test.tsx Outdated Show resolved Hide resolved
@bentoper bentoper force-pushed the feat/select-atom branch 4 times, most recently from 47cc7d8 to c42fa82 Compare August 11, 2021 16:31
@igorbrasileiro
Copy link
Contributor

igorbrasileiro commented Aug 11, 2021

@bentoper bentoper changed the title Feat: Adding select atom feat(store-ui): Adding select atom Aug 12, 2021
@bentoper bentoper requested a review from victorhmp August 12, 2021 17:54
@bentoper bentoper force-pushed the feat/select-atom branch 2 times, most recently from c8b466a to 984ff83 Compare August 16, 2021 14:10
@bentoper bentoper force-pushed the feat/select-atom branch 2 times, most recently from a9edb79 to be51f41 Compare August 16, 2021 19:35
@bentoper bentoper dismissed tlgimenes’s stale review August 17, 2021 17:38

PR opened on all stores without any breaking changes

@emersonlaurentino emersonlaurentino linked an issue Aug 18, 2021 that may be closed by this pull request
@bentoper bentoper force-pushed the feat/select-atom branch 2 times, most recently from 9114a56 to 3f57c34 Compare August 23, 2021 14:09
@bentoper bentoper merged commit 3276269 into master Aug 23, 2021
@bentoper bentoper deleted the feat/select-atom branch August 23, 2021 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Select
5 participants