chec / commercejs-nextjs-demo-store Goto Github PK

Commerce demo store built for the Jamstack. Built with Commerce.js, Next.js, and can be one-click deployed to Netlify. Includes product catalog, customer login, categories, variants, cart, checkout, payments (Stripe) order confirmation, and printable receipts.

Home Page: https://commercejs-demo-store.netlify.app/

License: BSD 3-Clause "New" or "Revised" License

JavaScript 80.25% Shell 0.28% SCSS 19.47%

headless nextjs netlify commercejs chec ecommerce open-source one-click-deploy serverless jamstack

commercejs-nextjs-demo-store's People

Contributors

Stargazers

Watchers

Forkers

jaepass philhawksworth garnerp mumer92 ollivr iamconsultoria mazurolegdev philabsto irahulcse asd339 ltfschoen slavakurilyak trishitapingolia kichloo darekkasata parshwa52 yashs911 corradiniste omkhulbe tacoelho abhi947223 sonam2905 ng29 thornm9 shibam17 drobiu aman-zishan parammittal16 nevergiv3up endernakamoto oshadabasnayake felipeeu sweetcherryvn cleong14 teleaziz sebstuart-banks nchen63 pedrsntana bamzi raialok michaeldpierce alysiawu triptych fukuru weiplanet dstuartelliott jasonadkison zacheryconverse virtuoushub rahulgopinathan mariatorrentedev begueradj lffranca michaelsupdegraff hnq90 nabeelhassan threadstonesecure cybernetics h0m3brew tomiwaale quicktoolbox immaruf hackthegist wpappdev seylantech harumuki hmz9343 aliarain doc22940 professionalsixoneg iamthenewking justinbhalla diogenespolanco elvisduru zahidaramai murilogteixeira soulvsn dudul-bdul st46593 zb2oby nathansom andreasoto jacoblee325 ddmunky beingsane sholamide andrewriveradev souravrao-31 itslonson flemon psychic89 whatshopmx work-stufff test-denar sgujje28 nolegenddeveloper paitoanderson gr-qft zizwar orlandoabreu

commercejs-nextjs-demo-store's Issues

Credit card payment field alignment is out at ~1000px wide screen

Add loading states

Add About page

Commerce.js example storefront

This is an open source storefront built using Chec’s headless architecture and the Commerce.js SDK. Commerce.js is a smarter, more efficient way to build eCommerce projects, and we hope you can see that as you explore the world custom eCommerce and build on top of, or pick apart this project. Everything in this project, from the storefront, to the cart, checkout and receipt, is 100% customizable. To build more custom eCommerce projects, using any frontend framework or platform, head over to commercejs.com.

After successful login, UI renders "... logged out"

This message shows after a successful login. Only after a hard refresh the account page with order details renders correctly

Test Netlify one-click deploy/CLI deploy

Login callback URL doesn't have route params provided, prevents login from working

Go to demo store when it has been statically generated (not yarn dev)
Click Login
Enter email and submit
Go to email, click link in email

Expected: Customer should be logged in and be shown customer section
Actual: login form shows again

https://nextjs.org/docs/routing/dynamic-routes#caveats

Netlify onboarding plugin not working

Added onboarding plugin to Netlify config, tested and worked fine
Somehow regressed and now onboarding screen does not show extra template.environment property to input env var

Add loading indicator while product display page is being loaded

Go to collections page
Click on a product

Notice there is a second or two while the product information loads that nothing happens in the UI. We could put a loading indicator somewhere to show that something is happening.

Cart bugfix

cart bug with retrieving new cart id

Product assets

product assets showing 2 image assets instead of 3

Add option to set up project with Chec CLI demo-store in setup steps

Product image previews not loading at the checkout

Review - Choose to bind methods with `this` or with arrow functions

Choose to bind methods with this or with arrow functions and keep it consistent across the whole codebase

Add Size to Checkout Page

Please add size (variations) to the checkout page

"Write a review" does nothing when you click on it

From the product view page. Rev 5606b20

Convert all component based architecture into functional components

Replace with hooks for fetching data, etc.

Ideally we split the templates up more before this happens, which is a separate issue: #160

Seeding is incomplete

Steps to reproduce:

Set CHEC_PUBLIC_KEY and CHEC_SECRET_KEY to your own keys in .env
Run yarn seed

It should run successfully and output something like this:

√ Completed seeding
Added:
  3 categories
  9 products
  27 assets
Done in 70.70s.

Run yarn dev

Actual behavior: no products are visible in the store.
Expected behavior: 9 products should be visible in the store.

Review/Use action types for store prod/cat in _app.js

use already created action types in _app.js for storing products and categories

Add GitHub action to run some automated checks

Example of some checks that we can run:

Linting (we'll need to add a linter and lint the project first)
yarn deploy - this is what Netlify/Vercel run to deploy the project

We often have pull requests merged that break the one-click deployment pipelines, but we don't discover this until that happens and somebody reports it (#136). This could be avoided by adding continuous integration checks for the deployment command, which is essentially the same locally as it is in the hosting provider.

Linting is a nice to have - the deployment command is the higher value addition for now.

Rename repo

nextjs-commercejs

Checkout on mobile - can't see cart

When on checkout page on mobile, can't see the cart recap which you should?

Show Shipping method validation error to customer

If a shipping method is not selected at the checkout the customer should be made aware of what they need to do to complete the checkout.

Proposed solutions:

Notification bar stating "Please select a shipping method to complete your order"
Highlight the Shipping Method field in red

Shipping validation error

Error should be more contextual
There shouldn't be an error in the first place

Centralise logic for redirecting to login when user is not logged in

On the account page, and the order view page, we have a verifyAuth method which runs when the component mounts. This checks that the user is logged in and redirects them away if not.

We can convert this into a HOC or something to centralise this logic.

Reviews error

Clicking on View all Reviews --> Takes to a page "An expected error has occurred."

Change `commerce.services.localeListCountries` to `commerce.services.localeListShippingCountries`

Change commerce.services.localeListCountries to commerce.services.localeListShippingCountries for better handling of populating available shipping countries
For more context #114 (comment)

Processing animation does not reset when there is a checkout error

Processing animation does not stop after/when shipping method alert is confirmed. Cannot progress with the order.

Bug/Adding second product triggers error

error:

{"error":{"type":"not_found","message":"This product doesn't have this variant option."},"status_code":404}

To reproduce:

go into a product detail page from the collection page, select a product and add a variant to cart (item is updated in cart)
go back to collection page and select another product to add to cart (item is updated in cart)
scroll down from current product detail page to a product card under suggested product and add that product to cart (item is not added to cart and triggers an error because previous initial state is not there)

Reason:

getStaticProps pages which are all the product detail pages in [permalink] are being executed at compile time and not able to get state from getInitialProps (different context)
ref: https://github.com/kirill-konshin/next-redux-wrapper#app-and-getserversideprops-or-getstaticprops-at-page-level
At this stage I'm not still not really sure how to go about hydrating the client state while directly navigating from one product detail page to another

Adding Table of Content to Readme.md

Hey @robbieaverill / @john-raymon / @jaepass

The idea for Adding Table of Content to Readme.md to add additional enhancement and readability to the docs file.

It will help users to have a better experience and will avoid unnecessary scrolling on the page.

I would like to work on this issue.

Cheers,

yarn build failed on netlify

After verifying that the demo store functions fine on http://localhost:3000, I followed the instructions in the README to deploy to Netlify, but it failed. Here is the log:

5:07:16 PM: Build ready to start
5:07:18 PM: build-image version: b0258b965567defc4a2d7e2f2dec2e00c8f73ad6
5:07:18 PM: build-image tag: v3.4.1
5:07:18 PM: buildbot version: 19f1164460e8d8241255cf2a71abc5e0e454118e
5:07:18 PM: Fetching cached dependencies
5:07:18 PM: Failed to fetch cache, continuing with build
5:07:18 PM: Starting to prepare the repo for build
5:07:19 PM: No cached dependencies found. Cloning fresh repo
5:07:19 PM: git clone https://github.com/ocmyusa/store
5:07:20 PM: Preparing Git Reference refs/heads/main
5:07:21 PM: Starting build script
5:07:21 PM: Installing dependencies
5:07:21 PM: Python version set to 2.7
5:07:23 PM: v12.18.0 is already installed.
5:07:23 PM: Now using node v12.18.0 (npm v6.14.4)
5:07:23 PM: Started restoring cached build plugins
5:07:23 PM: Finished restoring cached build plugins
5:07:24 PM: Attempting ruby version 2.7.1, read from environment
5:07:25 PM: Using ruby version 2.7.1
5:07:26 PM: Using PHP version 5.6
5:07:26 PM: 5.2 is already installed.
5:07:26 PM: Using Swift version 5.2
5:07:26 PM: Started restoring cached node modules
5:07:26 PM: Finished restoring cached node modules
5:07:26 PM: Installing NPM modules using NPM version 6.14.4
5:07:30 PM: npm WARN deprecated [email protected]: request has been deprecated, see https://github.com/request/request/issues/3142
5:07:34 PM: npm WARN deprecated [email protected]: this library is no longer supported
5:07:38 PM: npm WARN deprecated [email protected]: core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
5:07:39 PM: npm WARN deprecated [email protected]: Please see https://github.com/lydell/urix#deprecated
5:07:41 PM: npm WARN deprecated [email protected]: https://github.com/lydell/resolve-url#deprecated
5:07:41 PM: npm WARN deprecated [email protected]: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
5:07:42 PM: npm WARN deprecated [email protected]: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
5:08:08 PM: > [email protected] install /opt/build/repo/node_modules/node-sass
5:08:08 PM: > node scripts/install.js
5:08:08 PM: Downloading binary from https://github.com/sass/node-sass/releases/download/v4.14.1/linux-x64-72_binding.node
5:08:09 PM: Download complete
5:08:09 PM: Binary saved to /opt/build/repo/node_modules/node-sass/vendor/linux-x64-72/binding.node
5:08:09 PM: Caching binary to /opt/buildhome/.npm/node-sass/4.14.1/linux-x64-72_binding.node
5:08:09 PM: > [email protected] postinstall /opt/build/repo/node_modules/core-js
5:08:09 PM: > node -e "try{require('./postinstall')}catch(e){}"
5:08:09 PM: > [email protected] postinstall /opt/build/repo/node_modules/core-js-pure
5:08:09 PM: > node -e "try{require('./postinstall')}catch(e){}"
5:08:10 PM: > [email protected] postinstall /opt/build/repo/node_modules/node-sass
5:08:10 PM: > node scripts/build.js
5:08:10 PM: Binary found at /opt/build/repo/node_modules/node-sass/vendor/linux-x64-72/binding.node
5:08:10 PM: Testing binary
5:08:10 PM: Binary is fine
5:08:10 PM: > [email protected] postinstall /opt/build/repo/node_modules/swiper
5:08:10 PM: > echo "Love Swiper? Support Vladimir's work by donating or pledging on patreon: > https://patreon.com/vladimirkharlampidi
5:08:10 PM: "
5:08:10 PM: Love Swiper? Support Vladimir's work by donating or pledging on patreon: > https://patreon.com/vladimirkharlampidi
5:08:12 PM: npm notice created a lockfile as package-lock.json. You should commit this file.
5:08:12 PM: npm WARN notsup Unsupported engine for [email protected]: wanted: {"npm":"^3.0.0"} (current: {"node":"12.18.0","npm":"6.14.4"})
5:08:12 PM: npm WARN notsup Not compatible with your version of node/npm: [email protected]
5:08:12 PM: npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@~2.1.2 (node_modules/chokidar/node_modules/fsevents):
5:08:12 PM: npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
5:08:12 PM: npm WARN notsup Unsupported engine for [email protected]: wanted: {"node":"<8.10.0"} (current: {"node":"12.18.0","npm":"6.14.4"})
5:08:12 PM: npm WARN notsup Not compatible with your version of node/npm: [email protected]
5:08:12 PM: npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@^1.2.7 (node_modules/watchpack-chokidar2/node_modules/chokidar/node_modules/fsevents):
5:08:12 PM: npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
5:08:12 PM: npm WARN [email protected] requires a peer of typescript@>=2.8.0 || >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta but none is installed. You must install peer dependencies yourself.
5:08:12 PM: npm WARN [email protected] No repository field.
5:08:12 PM: added 1533 packages from 536 contributors and audited 1536 packages in 44.814s
5:08:13 PM: 63 packages are looking for funding
5:08:13 PM:   run `npm fund` for details
5:08:13 PM: found 4 vulnerabilities (2 low, 2 high)
5:08:13 PM:   run `npm audit fix` to fix them, or `npm audit` for details
5:08:13 PM: NPM modules installed
5:08:13 PM: Started restoring cached go cache
5:08:13 PM: Finished restoring cached go cache
5:08:13 PM: go version go1.14.4 linux/amd64
5:08:13 PM: go version go1.14.4 linux/amd64
5:08:13 PM: Installing missing commands
5:08:13 PM: Verify run directory
5:08:15 PM: 
5:08:15 PM: ┌─────────────────────────────┐
5:08:15 PM: │        Netlify Build        │
5:08:15 PM: └─────────────────────────────┘
5:08:15 PM: 
5:08:15 PM: ❯ Version
5:08:15 PM:   @netlify/build 5.1.0
5:08:15 PM: 
5:08:15 PM: ❯ Flags
5:08:15 PM:   deployId: 5f9caab436a6754286d1116f
5:08:15 PM:   mode: buildbot
5:08:15 PM: 
5:08:15 PM: ❯ Current directory
5:08:15 PM:   /opt/build/repo
5:08:15 PM: 
5:08:15 PM: ❯ Config file
5:08:15 PM:   /opt/build/repo/netlify.toml
5:08:15 PM: 
5:08:15 PM: ❯ Context
5:08:15 PM:   production
5:08:15 PM: 
5:08:15 PM: ❯ Loading plugins
5:08:15 PM:    - [email protected] from netlify.toml and package.json
5:08:15 PM: 
5:08:15 PM: ┌──────────────────────────────────────────────────────┐
5:08:15 PM: │ 1. onPreBuild command from netlify-plugin-onboarding │
5:08:15 PM: └──────────────────────────────────────────────────────┘
5:08:15 PM: 
5:08:15 PM: Cloning into '/opt/build/repo/node_modules/netlify-plugin-onboarding/template'...
5:08:26 PM: > [email protected] postinstall /opt/build/repo/node_modules/netlify-plugin-onboarding/template/node_modules/core-js
5:08:26 PM: > node -e "try{require('./postinstall')}catch(e){}"
5:08:27 PM: > [email protected] postinstall /opt/build/repo/node_modules/netlify-plugin-onboarding/template/node_modules/ejs
5:08:27 PM: > node ./postinstall.js
5:08:27 PM: npm WARN optional SKIPPING OPTIONAL DEPENDENCY: [email protected] (node_modules/chokidar/node_modules/fsevents):
5:08:27 PM: npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
5:08:27 PM: npm WARN optional SKIPPING OPTIONAL DEPENDENCY: [email protected] (node_modules/fsevents):
5:08:27 PM: npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
5:08:27 PM: added 528 packages from 405 contributors and audited 533 packages in 10.866s
5:08:27 PM: 6 packages are looking for funding
5:08:27 PM:   run `npm fund` for details
5:08:27 PM: found 9 vulnerabilities (7 low, 2 high)
5:08:27 PM:   run `npm audit fix` to fix them, or `npm audit` for details
5:08:28 PM: > [email protected] build /opt/build/repo/node_modules/netlify-plugin-onboarding/template
5:08:28 PM: > eleventy --input src --output dist
5:08:28 PM: Writing dist/index.html from ./src/index.md.
5:08:28 PM: Wrote 1 file in 0.11 seconds (v0.11.0)
5:08:28 PM: 
5:08:28 PM: (netlify-plugin-onboarding onPreBuild completed in 13.1s)
5:08:28 PM: 
5:08:28 PM: ┌────────────────────────────────────┐
5:08:28 PM: │ 2. build.command from netlify.toml │
5:08:28 PM: └────────────────────────────────────┘
5:08:28 PM: 
5:08:28 PM: $ yarn deploy
5:08:28 PM: bash: yarn: command not found
5:08:28 PM: 
5:08:28 PM: ┌─────────────────────────────┐
5:08:28 PM: │   "build.command" failed    │
5:08:28 PM: └─────────────────────────────┘
5:08:28 PM: 
5:08:28 PM:   Error message
5:08:28 PM:   Command failed with exit code 127: yarn deploy
5:08:28 PM: 
5:08:28 PM:   Error location
5:08:28 PM:   In build.command from netlify.toml:
5:08:28 PM:   yarn deploy
5:08:28 PM: 
5:08:28 PM:   Resolved config
5:08:28 PM:   build:
5:08:28 PM:     command: yarn deploy
5:08:28 PM:     commandOrigin: config
5:08:28 PM:     publish: /opt/build/repo/out
5:08:28 PM:   plugins:
5:08:28 PM:     - inputs: {}
5:08:28 PM:       origin: config
5:08:28 PM:       package: netlify-plugin-onboarding
5:08:28 PM: Caching artifacts
5:08:28 PM: Started saving node modules
5:08:28 PM: Finished saving node modules
5:08:28 PM: Started saving build plugins
5:08:28 PM: Finished saving build plugins
5:08:28 PM: Started saving pip cache
5:08:29 PM: Finished saving pip cache
5:08:29 PM: Started saving emacs cask dependencies
5:08:29 PM: Finished saving emacs cask dependencies
5:08:29 PM: Started saving maven dependencies
5:08:29 PM: Finished saving maven dependencies
5:08:29 PM: Started saving boot dependencies
5:08:29 PM: Finished saving boot dependencies
5:08:29 PM: Started saving go dependencies
5:08:29 PM: Finished saving go dependencies
5:08:31 PM: Build failed due to a user error: Build script returned non-zero exit code: 2
5:08:31 PM: Failing build: Failed to build site
5:08:31 PM: Failed during stage 'building site': Build script returned non-zero exit code: 2
5:08:31 PM: Finished processing build request in 1m13.431100281s

Could it be because the GitHub repo is private?

Add code block showing repo structure with descriptions of main folders/files in README

Totals issue at checkout and receipt page

Review - Rename methods and callbacks event handlers

Following React's convention to name methods on components and the callbacks passed as props to handle... and on...

Added missing and Fixed inconsistent links in ReadMe docs

Hi @robbieaverill / @john-raymon / @jaepass

As reading the ReadMe docs found some inconsistencies like

Missing links on Next.js in some of the major areas
Missing links of some technologies used in Tech stack ...

I would like to work on this

Thanks

Product detail container not responsive

product detail container out of view in mobile -> make responsive

About page on home banner = 404

https://commercejs-demo-store.netlify.app/about ==> 404.
Either create new page or redirect to commercejs.com #

Same thing happens when clicking on Find out More --> perhaps link to instagram?

Menu Popup on devices with resolution less than 720px ,bg color looks little aweful, need to optimise that

Command failed with exit code 1: yarn deploy

Hello! Building with Netlify, I got error when website is deployed :

1:22:20 PM: ┌─────────────────────────────┐
1:22:20 PM: │        Netlify Build        │
1:22:20 PM: └─────────────────────────────┘
1:22:20 PM: 
1:22:20 PM: ❯ Version
1:22:20 PM:   @netlify/build 3.1.10
1:22:20 PM: 
1:22:20 PM: ❯ Flags
1:22:20 PM:   deployId: 5f2d3943799009008fb82918
1:22:20 PM:   mode: buildbot
1:22:20 PM:   timersFile: /tmp/substage_times.txt
1:22:20 PM: 
1:22:20 PM: ❯ Current directory
1:22:20 PM:   /opt/build/repo
1:22:20 PM: 
1:22:20 PM: ❯ Config file
1:22:20 PM:   /opt/build/repo/netlify.toml
1:22:20 PM: 
1:22:20 PM: ❯ Context
1:22:20 PM:   production
1:22:20 PM: 
1:22:20 PM: ❯ Loading plugins
1:22:20 PM:    - [email protected] from netlify.toml and package.json
1:22:20 PM: 
1:22:20 PM: ┌──────────────────────────────────────────────────────┐
1:22:20 PM: │ 1. onPreBuild command from netlify-plugin-onboarding │
1:22:20 PM: └──────────────────────────────────────────────────────┘
1:22:20 PM: 
1:22:20 PM: 
1:22:20 PM: (netlify-plugin-onboarding onPreBuild completed in 5ms)
1:22:20 PM: 
1:22:20 PM: ┌────────────────────────────────────┐
1:22:20 PM: │ 2. build.command from netlify.toml │
1:22:20 PM: └────────────────────────────────────┘
1:22:20 PM: 
1:22:20 PM: $ yarn deploy
1:22:21 PM: yarn run v1.22.4
1:22:21 PM: $ next build && next export
1:22:21 PM: Browserslist: caniuse-lite is outdated. Please run the following command: `yarn upgrade`
1:22:21 PM: Warning: No build cache found. Please configure build caching for faster rebuilds. Read more: https://err.sh/next.js/no-cache
1:22:21 PM: Creating an optimized production build...
1:22:22 PM: Warning: Built-in CSS support is being disabled due to custom CSS configuration being detected.
1:22:22 PM: See here for more info: https://err.sh/next.js/built-in-css-disabled
1:22:48 PM: Compiled successfully.
1:22:48 PM: 
1:22:48 PM: Automatically optimizing pages...
1:22:48 PM: Warning: You have opted-out of Automatic Static Optimization due to `getInitialProps` in `pages/_app`.
1:22:48 PM: Read more: https://err.sh/next.js/opt-out-auto-static-optimization
1:22:50 PM: > Build error occurred
1:22:50 PM: TypeError: Cannot read property 'map' of undefined
1:22:50 PM:     at getStaticPaths (/opt/build/repo/.next/server/static/HEPUvl8hVstwABV3n7nTb/pages/product/[permalink].js:2546:26)
1:22:50 PM:     at processTicksAndRejections (internal/process/task_queues.js:97:5)
1:22:50 PM:     at async buildStaticPaths (/opt/build/repo/node_modules/next/dist/build/utils.js:18:80)
1:22:50 PM:     at async Object.isPageStatic (/opt/build/repo/node_modules/next/dist/build/utils.js:25:542) {
1:22:50 PM:   type: 'TypeError'
1:22:50 PM: }
1:22:50 PM: error Command failed with exit code 1.
1:22:50 PM: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
1:22:50 PM: 
1:22:50 PM: ┌─────────────────────────────┐
1:22:50 PM: │   "build.command" failed    │
1:22:50 PM: └─────────────────────────────┘
1:22:50 PM: 
1:22:50 PM:   Error message
1:22:50 PM:   Command failed with exit code 1: yarn deploy
1:22:50 PM: 
1:22:50 PM:   Error location
1:22:50 PM:   In build.command from netlify.toml:
1:22:50 PM:   yarn deploy
1:22:50 PM: 
1:22:50 PM:   Resolved config
1:22:50 PM:   build:
1:22:50 PM:     command: yarn deploy
1:22:50 PM:     commandOrigin: config
1:22:50 PM:     environment:
1:22:50 PM:       - CHEC_PUBLIC_KEY
1:22:50 PM:     publish: /opt/build/repo/out
1:22:50 PM:   plugins:
1:22:50 PM:     - inputs: {}
1:22:50 PM:       origin: config
1:22:50 PM:       package: netlify-plugin-onboarding
1:22:50 PM: Caching artifacts
1:22:50 PM: Started saving node modules
1:22:50 PM: Finished saving node modules
1:22:50 PM: Started saving build plugins
1:22:50 PM: Finished saving build plugins
1:22:50 PM: Started saving yarn cache
1:22:50 PM: Finished saving yarn cache
1:22:50 PM: Started saving pip cache
1:22:50 PM: Finished saving pip cache
1:22:50 PM: Started saving emacs cask dependencies
1:22:50 PM: Finished saving emacs cask dependencies
1:22:50 PM: Started saving maven dependencies
1:22:50 PM: Finished saving maven dependencies
1:22:50 PM: Started saving boot dependencies
1:22:50 PM: Finished saving boot dependencies
1:22:50 PM: Started saving go dependencies
1:22:50 PM: Finished saving go dependencies
1:22:52 PM: Error running command: Build script returned non-zero exit code: 1
1:22:52 PM: Failing build: Failed to build site
1:22:53 PM: Failed during stage 'building site': Build script returned non-zero exit code: 1
1:22:53 PM: Finished processing build request in 1m11.744157387s

My key is in the right place :

Add a contribution outline doc

Add category in dashboard messes up a Link in CategoryBanner.js

If I add a new category and refresh I get the error "Error: Failed prop type: The prop href expects a string or object in <Link>, but got undefined instead."

I traced it by luck to <Link href={item.link}> in components/homepage/CategoryBanner.js.

The error goes away if I remove the new category I created in the dashboard, or if I change the Link to <Link href="/">

Delete unused components/pages

delete any unused components/pages
delete unused code throughout project

Better error handling when categories are deleted in the backend

Scenario:

When a user clones the project, seeds the sample data and sets up the app locally, he/she deletes the sample categories
The hard coded categories data here https://github.com/chec/commercejs-nextjs-demo-store/blob/master/lib/collections.js does not match up and the UI cannot be rendered

Solutions:

Better error handling
Get rid of hardcoded collections data and somehow output dynamically or remove from UI where it is used in home page categories banner and sidebar in shop pages and pdp (added categories feature image and product count)

Bug/Store data not persisting going from one getStaticProps product display page to another

Store data like items in cart not persisted going from one getStaticProps "product detail page" to another after next-wrapper-redux udpate

Add discount functionality in checkout

No option to NOT invert logo

This project is really nice to get started with next.js and commerce.js.
I would like to contribute all I can, but I'm a noob. Hopefully I can contribute as I learn, but seemingly not yet.
I have a proposal, and it might be a small and easy fix, but I can't do it.

It is indeed stylish that the header is transparent and inverts color of the fonts/logo/shopping cart when you're all the way up the page.
But, inverting the colors of the logo might not be suitable as I guess a brand would like their main logo consistent in colors.
Hence, this proposal to have an option to not invert the logo.

in components/common/Header.js you have your logo and logo-container:

      <div className="logo-container">
        <img
          src={`/icon/${showMobileMenu ? 'cross' : 'menu'}.svg`}
          onClick={this.toggleMobileMenu}
          className="w-32 mr-1 d-block d-sm-none"
          alt="Menu icon"
        />
        <Link href="/">
          <a>
            <img
              src="/images/commerce.svg"
              className="logo cursor-pointer"
              alt="Logo"
            />
          </a>
        </Link>
      </div>

In the same file, there is this code which I believe is responsible for the header invert:

    <div
      ref={this.header}
      className={`d-flex header align-items-center justify-content-between position-relative ${
        transparent ? '' : 'invert'
      }`}
    >

Anyone have suggestions on this fix?

Here's a screenshot of what I see.

Please advise.

Deploy to Netlify Button Fails

Hi, just FYI, the Deploy to Netlify button leads to deployment failures in Netlify.