-
Notifications
You must be signed in to change notification settings - Fork 14
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Preview is readyThis pull request generated a Preview👀 Preview: https://preview-34--gatsby.preview.vtex.app |
Lighthouse ReportsHere are the Lighthouse reports of this Pull Request📝 Based on commit 791e82d
|
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!
Please take a look at my other comment.
0102e4b
to
37aed14
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.
Nice 👏
What's the purpose of this pull request?
This PR is part of the Component Placement project. Replaces
Card
forProductCard
How does it work?
ProductCard
component from FSUI instead ofCard
ProductCard
propactionabled
toactionable
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:Checklist
You may erase this after checking them all ;)
ProductCard
component vtex/faststore#1272 is merged, update faststore/ui version on stores and merge those PRs as wellCHANGELOG.md
at the beginning of its due section. The latest version should comes first.CHANGELOG.md
. E.g., New items in thepull_request_template.md
(#12)