saleor / saleor-storefront

A GraphQL-powered, NextJs-based, PWA storefront for Saleor. IMPORTANT: This project is [DEPRECATED] in favor of saleor/react-storefront soon to become our default demo and storefront starter pack.
https://demo.saleor.io/
BSD 3-Clause "New" or "Revised" License
770 stars 672 forks source link

Feature/saleor 1564 Add next.js to the Storefront #960

Closed piotrgrundas closed 3 years ago

piotrgrundas commented 3 years ago

I want to merge this change because...

Add next.js to the Storefront

Screenshots

Pull Request Checklist

  1. [x] All visible strings are translated with proper context.
  2. [x] All data-formatting is locale-aware (dates, numbers, and so on).
  3. [x] The changes are tested.
  4. [x] The code is documented (docstrings, project documentation).
  5. [x] Changes are mentioned in the changelog.

Test Environment Config

API_URI=https://master.staging.saleor.rocks/graphql/

patrys commented 3 years ago

Task linked: SALEOR-1564 Add next.js to the Storefront

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 62629fc416c67592aba8260d7c6a4922e31c5e6d into 1abbf68c3ae1f8d83d1fbce025e46b653899c3e1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging bb5eeab9d81a837ccdbaaebec79eda1aa69a6c33 into 1abbf68c3ae1f8d83d1fbce025e46b653899c3e1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging e5b6c223539c43ac91e54f3260b3997ea25552f3 into 1abbf68c3ae1f8d83d1fbce025e46b653899c3e1 - view on LGTM.com

new alerts:

krzysztofwolski commented 3 years ago

Minor things to change, rest is looking good 👍

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 218f99d95007221712dcfc507ccc4c89ba80ef3a into 1abbf68c3ae1f8d83d1fbce025e46b653899c3e1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 062a7dc447b867162deb4ca49944d9a0431bc385 into 1abbf68c3ae1f8d83d1fbce025e46b653899c3e1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 2032ff17f3ddb89fdf7dba12b194dc0857cd23d9 into 1abbf68c3ae1f8d83d1fbce025e46b653899c3e1 - view on LGTM.com

new alerts: