commerce icon indicating copy to clipboard operation
commerce copied to clipboard

Add review feature and disable it for BC stores

Open powerwlsl opened this issue 3 years ago • 4 comments

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!

powerwlsl avatar Aug 10 '22 17:08 powerwlsl

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

A member of the Team first needs to authorize it.

vercel[bot] avatar Aug 10 '22 17:08 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)

vercel[bot] avatar Aug 10 '22 17:08 vercel[bot]

@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.

cond0r avatar Aug 19 '22 08:08 cond0r

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.

powerwlsl avatar Aug 19 '22 22:08 powerwlsl

Hey there! Thank you for your contribution. We have decided to take Next.js Commerce in a new direction and will be closing out current PRs and issues due to this change. Please see this PR for more details: https://github.com/vercel/commerce/pull/966

P.S. we would love to see a BigCommerce version of the new v2 and happy to chat more about this 🙏

leerob avatar Apr 18 '23 02:04 leerob