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

refactor: Extract CartItem from the starter to @faststore/ui #198

Merged
merged 1 commit into from
Sep 13, 2022

Conversation

filipewl
Copy link
Contributor

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\
@filipewl filipewl self-assigned this Sep 12, 2022
@vercel
Copy link

vercel bot commented Sep 12, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
gatsby-store-storybook ✅ Ready (Inspect) Visit Preview Sep 12, 2022 at 11:55PM (UTC)

@vtex-sites
Copy link

vtex-sites bot commented Sep 13, 2022

Lighthouse Reports

Here are the Lighthouse reports of this Pull Request

📝 Based on commit 9d5f6d5

Lighthouse Report by page
📎   /
📎   /apple-magic-mouse/p
📎   /office

@filipewl filipewl requested a review from a team September 13, 2022 00:09
@filipewl filipewl changed the title refactor: Extract CartItem from the starter to @faststore/ui (#239) refactor: Extract CartItem from the starter to @faststore/ui Sep 13, 2022
@filipewl filipewl marked this pull request as ready for review September 13, 2022 00:16
@filipewl
Copy link
Contributor Author

Tested on https://preview-198--gatsby.preview.vtex.app/apple-magic-mouse-99988212/p and it looks fine!

CleanShot 2022-09-12 at 21 11 12@2x

Merging this ahead since it's been approved in https://github.com/vtex-sites/nextjs.store/pull/239.

@filipewl filipewl merged commit 83d135c into main Sep 13, 2022
@filipewl filipewl deleted the refactor/fs-361-cart-item branch September 13, 2022 00:18
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.

1 participant