chec / commercejs-nextjs-demo-store

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.
https://commercejs-demo-store.netlify.app/
BSD 3-Clause "New" or "Revised" License
1.07k stars 206 forks source link

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

Closed jaepass closed 4 years ago

jaepass commented 4 years ago

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

robbieaverill commented 4 years ago

I might suggest we refer to "product detail pages" or "product details view" or something rather than PDP - I think "PDP" is a bit of a siloed term we've ended up adopting, but it's not clear what it means to anyone else (including me and Guy!)

robbieaverill commented 4 years ago

Regression from #43

jaepass commented 4 years ago

yea that's right, will look into refactoring the page level getStaticProps https://github.com/kirill-konshin/next-redux-wrapper#app-and-getserversideprops-or-getstaticprops-at-page-level

jaepass commented 4 years ago

so have gone through the docs and issues raised but still stumped on how to merge both server and client states or hydrate correctly. is getInitialProps() still compatible with the latest next-wrapper-redux upgrade? https://github.com/kirill-konshin/next-redux-wrapper/pull/196#