FormidableLabs / nextjs-sanity-fe

NextJS Demo site with Sanity CMS
https://nextjs-sanity.formidable.dev/
29 stars 6 forks source link

Fix variant type so tests pass #172

Closed sarmeyer closed 1 year ago

sarmeyer commented 1 year ago

This fix allows the tests to pass, but there is more that needs to be done to ensure the rest of the site loads the products in correctly.

We were using variants to represent all products, so when I moved them to live within a product document type and not next to it, the way we access all variants broke.

I'm going to keep working on updating the rest of the queries and manually update the data

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated
nextjs-sanity-fe ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 20, 2023 at 6:54PM (UTC)
luisabarca commented 1 year ago

LGTM!

Screenshot 2023-01-20 at 13 47 17
sarmeyer commented 1 year ago

going to revert the original variant changes and update our desk layout instead!