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: Uses ProductCard component from FSUI #34

Merged
merged 13 commits into from
May 17, 2022

Conversation

hellofanny
Copy link
Contributor

@hellofanny hellofanny commented May 10, 2022

What's the purpose of this pull request?

This PR is part of the Component Placement project. Replaces Card for ProductCard

How does it work?

  • Uses ProductCard component from FSUI instead of Card
  • Renames ProductCard prop actionabled to actionable

How to test it?

  • Nothing should change visually on the component.

  • Check on the preview if the ProductCard is working fine as before.

  • The ProductCard component appear on the Homepage and PLP:

Homepage - ProductShelf
image
Homepage
image
PLP - Product Listing Page
plp

Checklist

You may erase this after checking them all ;)

@vercel
Copy link

vercel bot commented May 10, 2022

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

Name Status Preview Updated
gatsby-store-storybook ✅ Ready (Inspect) Visit Preview May 17, 2022 at 0:42AM (UTC)

@hellofanny hellofanny self-assigned this May 10, 2022
@vtex-sites
Copy link

vtex-sites bot commented May 10, 2022

Preview is ready

This pull request generated a Preview

👀   Preview: https://preview-34--gatsby.preview.vtex.app
🔬   Go deeper by inspecting the Build Logs
📝   based on commit 791e82d

@vtex-sites
Copy link

vtex-sites bot commented May 13, 2022

Lighthouse Reports

Here are the Lighthouse reports of this Pull Request

📝 Based on commit 791e82d

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

Copy link
Contributor

@filipewl filipewl left a comment

Choose a reason for hiding this comment

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

LGTM!

Please take a look at my other comment.

yarn.lock Outdated Show resolved Hide resolved
@hellofanny hellofanny force-pushed the feat/FSSS-2461-replace-product-card branch from 0102e4b to 37aed14 Compare May 16, 2022 23:43
Copy link
Collaborator

@renatamottam renatamottam left a comment

Choose a reason for hiding this comment

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

Nice 👏

@hellofanny hellofanny merged commit 791e82d into main May 17, 2022
@hellofanny hellofanny deleted the feat/FSSS-2461-replace-product-card branch May 17, 2022 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore General tasks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants