saleor 1563 nextjs integration

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

769 stars 672 forks source link

saleor 1563 nextjs integration #955

Closed piotrgrundas closed 3 years ago

piotrgrundas commented 4 years ago

I want to merge this change because...

Screenshots

Pull Request Checklist

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

Test Environment Config

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

lgtm-com[bot] commented 4 years ago

This pull request introduces 6 alerts when merging 7dd3f40e043d83f1ef84fc3c48e583feb1db873f into d493e1b50a2b6c290ffc8a53c424f7b5330b8d0a - view on LGTM.com

new alerts:

6 for Unused variable, import, function or class

lgtm-com[bot] commented 4 years ago

This pull request introduces 9 alerts when merging 270f6582c9b6e085a3d3879d0d05e15da0132d7b into d493e1b50a2b6c290ffc8a53c424f7b5330b8d0a - view on LGTM.com

new alerts:

8 for Unused variable, import, function or class
1 for Assignment to constant

lgtm-com[bot] commented 4 years ago

This pull request introduces 8 alerts when merging 8764d9496f078dea0338a68a69192d706bfbabd6 into d493e1b50a2b6c290ffc8a53c424f7b5330b8d0a - view on LGTM.com

new alerts:

8 for Unused variable, import, function or class

lgtm-com[bot] commented 4 years ago

This pull request introduces 4 alerts when merging 44c0c6645767780b3cf6f43aacc1ef9035dcc186 into d493e1b50a2b6c290ffc8a53c424f7b5330b8d0a - view on LGTM.com

new alerts:

2 for Unused variable, import, function or class
1 for Unreachable statement
1 for Useless conditional

lgtm-com[bot] commented 4 years ago

This pull request introduces 3 alerts when merging b3698c5691b1f1f5954bf53d7e1116760624f712 into d493e1b50a2b6c290ffc8a53c424f7b5330b8d0a - view on LGTM.com

new alerts:

2 for Useless conditional
1 for Unused variable, import, function or class

lgtm-com[bot] commented 4 years ago

This pull request introduces 2 alerts when merging cbe39521df7ff9c2bf2ff76c290e76d75b9c22a6 into 5aaa9f7c095b15c34f356fd20ef064e4e84702ed - view on LGTM.com

new alerts:

2 for Useless conditional

lgtm-com[bot] commented 4 years ago

This pull request introduces 2 alerts when merging 3e8a4ab8bd96ab52ae6418522cd3ae4ef698fdca into 5aaa9f7c095b15c34f356fd20ef064e4e84702ed - view on LGTM.com

new alerts:

2 for Useless conditional

lgtm-com[bot] commented 4 years ago

This pull request introduces 2 alerts when merging e011af515b634822f5d8ed8c91a7828faeabbcd0 into 5aaa9f7c095b15c34f356fd20ef064e4e84702ed - view on LGTM.com

new alerts:

2 for Useless conditional