-
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
refactor: Extract CartItem
component from the starter
#1461
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 c5aba36:
|
96e35ca
to
c5aba36
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.
LGTM
Extract the `CartItem` component from our starter to `@faststore/ui`. Visit https://sfj-4a90f81--nextjs.preview.vtex.app/4k-philips-monitor-99988213/p and add a product to the cart. A `CartItem` component will be rendered inside the mini-cart sidebar. It should look and work the same as before. <img src="https://user-images.githubusercontent.com/381395/189658104-c64012ee-dd16-4ce0-a2bc-4535e8bec5d5.gif" width="50%" alt="Before and after changes" /> - https://github.com/vtex/faststore/pull/1461\
## What's the purpose of this pull request? Extract the `CartItem` component from our starter to `@faststore/ui`. ## How to test it? Visit https://sfj-4a90f81--nextjs.preview.vtex.app/4k-philips-monitor-99988213/p and add a product to the cart. A `CartItem` component will be rendered inside the mini-cart sidebar. It should look and work the same as before. <img src="https://user-images.githubusercontent.com/381395/189658104-c64012ee-dd16-4ce0-a2bc-4535e8bec5d5.gif" width="50%" alt="Before and after changes" /> ## References - vtex/faststore#1461
## What's the purpose of this pull request? Extract the `CartItem` component from our starter to `@faststore/ui`. ## How to test it? Visit https://sfj-4a90f81--nextjs.preview.vtex.app/4k-philips-monitor-99988213/p and add a product to the cart. A `CartItem` component will be rendered inside the mini-cart sidebar. It should look and work the same as before. <img src="https://user-images.githubusercontent.com/381395/189658104-c64012ee-dd16-4ce0-a2bc-4535e8bec5d5.gif" width="50%" alt="Before and after changes" /> ## References - https://github.com/vtex/faststore/pull/1461\
What's the purpose of this pull request?
Extract the
CartItem
component from our starter to@faststore/ui
.How to test it?
Run
![CleanShot 2022-09-12 at 08 28 54@2x](https://user-images.githubusercontent.com/381395/189643277-522a16b5-a9ba-4502-a0d0-d0cbc6e4534b.png)
yarn storybook
and check the newCartItem
entry at http://localhost:6006/?path=/story/molecules-cartitem--default.Starters Deploy Preview
Check https://sfj-4a90f81--nextjs.preview.vtex.app/4k-philips-monitor-99988213/p from https://github.com/vtex-sites/nextjs.store/pull/239. Add a product to the min-cart and you should see the
CartItem
being rendered in the sidebar.References