-
Notifications
You must be signed in to change notification settings - Fork 63
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
Conversation
✔️ 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 |
ed6ddff
to
b92bee1
Compare
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:
|
There was a problem hiding this 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
46ca71c
to
da7957e
Compare
3f8a8f2
to
271355e
Compare
6f804c6
to
bbde42c
Compare
There was a problem hiding this 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!
There was a problem hiding this 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?!
47cc7d8
to
c42fa82
Compare
Hey the preview seems not to be working https://deploy-preview-876--storeui.netlify.app/?path=/docs/atoms-select--default-select |
15aa23b
to
daf2a59
Compare
c8b466a
to
984ff83
Compare
a9edb79
to
be51f41
Compare
be51f41
to
d8e9530
Compare
PR opened on all stores without any breaking changes
d8e9530
to
0c6bf24
Compare
9114a56
to
3f57c34
Compare
- extending the props from a different base
474a3a5
to
9a8546a
Compare
What's the purpose of this pull request?
Adds the
Select
atom tostore-ui