Vercel from Commerce Repository

MIT License

Open
Add review feature and disable it for BC stores #814

Issue Opened
Powerwlsl Opened Issue On Aug 10th 2022, 5:51

Powerwlsl

Currently reviews and ratings are hard-coded and we want to disable it for the BC stores. Let me know if there are more steps when adding a feature!

Comment
Vercel[bot] commented on 3 months ago

Vercel[bot]

@powerwlsl is attempting to deploy a commit to the Vercel Solutions Team on Vercel.

A member of the Team first needs to authorize it.

Comment
Vercel[bot] commented on 3 months ago

Vercel[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

| Name | Status | Preview | Updated | | :--- | :----- | :------ | :------ | | commerce-bigcommerce | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-commerce | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-commercejs | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-kibocommerce | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-local | ✅ Ready (Inspect) | Visit Preview | Aug 10, 2022 at 5:52PM (UTC) | | commerce-ordercloud | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-pbt01 | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-saleor | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-sfcc | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-shopify | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-site | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-spree | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-swell | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) | | commerce-vendure | ❌ Failed (Inspect) | | Aug 10, 2022 at 5:52PM (UTC) |

Comment
Cond0r commented on 3 months ago

Cond0r

@powerwlsl The site example is more to show the functionality of the commerce package & providers. So in forks, anyone can change the site how is desired; no need to enable or disable it like this. And with this feature since is not implemented in any provider, you can safely remove it completely from your fork.

Comment
Powerwlsl commented on 3 months ago

Powerwlsl

Hi @cond0r! I understand what you meant and I think that would also work. But we came up with this idea because we wanted to make a default store for BC users less confusing by removing non functioning components. But if the Vercel team doesn't like our approach, we can follow as you said.

About Repository

Next.js Commerce
Language: TypeScript
Star: 6536
Fork: 2283
Watchers: 6536
Open Issues: 155
https://github.com/vercel/commerce
MIT License
Last updated: 05 Dec 2022
Tropics: