Add: storybook + seo component + gutneberg style package #65
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/rtcamp/wp-decoupled/n3leu30cu |
|
@bilouStrike try not to overdo things and keep PRs seperate . its faster and easier to manage things this way |
| "prettier": "^2.2.1", | ||
| "serialize-javascript": "^2.1.2" | ||
| } | ||
| "name": "wp-decoupled", |
There was a problem hiding this comment.
dont format the package.json
| ) | ||
| )} | ||
| <> | ||
| <NextSeo |
There was a problem hiding this comment.
Please add a todo to make the seo data dynamic, that would later come from WordPress
| @@ -4,45 +4,52 @@ import client from '../src/apollo/ApolloClient'; | |||
| import AddToCartButton from '../src/components/cart/AddToCartButton'; | |||
| import Hero from '../src/components/home/Hero'; | |||
| import { PRODUCTS_QUERY } from '../src/queries'; | |||
| import { NextSeo } from 'next-seo'; | |||
|
|
|||
| const NewProducts = ({ products }) => { | |||
| return ( | |||
There was a problem hiding this comment.
Please add Prop types for products, using prop-types npm package
pages/index.js
Outdated
| <span className="product-link"> | ||
| <img | ||
| className="product-image" | ||
| src={item.image.sourceUrl} |
There was a problem hiding this comment.
use optional chaining
item?.image?.sourceUrl
| "@storybook/addon-links": "^6.1.14", | ||
| "@storybook/react": "^6.1.14", | ||
| "@wordpress/base-styles": "^3.3.0", |
There was a problem hiding this comment.
Where are we using this package?
@wordpress/base-styles, we need to import it in sass file
… into setup-packages pul merge
This PR aims to close #60 #53