Correct Variant Added to Cart

components/product/ProductView/ProductView.tsx

@@ -11,7 +11,7 @@ import { Button, Container, Text } from '@components/ui'
 import usePrice from '@framework/product/use-price'
 import { useAddItem } from '@framework/cart'
 
-import { getCurrentVariant, SelectedOptions } from '../helpers'
+import { getVariant, SelectedOptions } from '../helpers'
 import WishlistButton from '@components/wishlist/WishlistButton'
 
 interface Props {
@@ -27,23 +27,24 @@ const ProductView: FC<Props> = ({ product }) => {
     baseAmount: product.price.retailValue,
     currencyCode: product.price.currencyCode!,
   })
-
   const { openSidebar } = useUI()
-
   const [loading, setLoading] = useState(false)
   const [choices, setChoices] = useState<SelectedOptions>({
     size: null,
     color: null,
   })
 
-  // const variant = getCurrentVariant(product, choices) || product.variants[0]
-  console.log('PRODUCT VIEW', product)
+  // Select the correct variant based on choices
+  const variant = getVariant(product, choices)
+
   const addToCart = async () => {
     setLoading(true)
     try {
       await addItem({
         productId: Number(product.id),
-        variantId: Number(product.variants[0].id), // TODO(bc) send the correct variant
+        variantId: variant
+          ? Number(variant.id)
+          : Number(product.variants[0].id),
       })
       openSidebar()
       setLoading(false)
@@ -108,11 +109,14 @@ const ProductView: FC<Props> = ({ product }) => {
                 <h2 className="uppercase font-medium">{opt.displayName}</h2>
                 <div className="flex flex-row py-4">
                   {opt.values.map((v, i: number) => {
-                    const active = (choices as any)[opt.displayName]
+                    const active = (choices as any)[
+                      opt.displayName.toLowerCase()
+                    ]
+
                     return (
                       <Swatch
                         key={`${opt.id}-${i}`}
-                        active={v.label === active}
+                        active={v.label.toLowerCase() === active}
                         variant={opt.displayName}
                         color={v.hexColors ? v.hexColors[0] : ''}
                         label={v.label}
@@ -120,7 +124,7 @@ const ProductView: FC<Props> = ({ product }) => {
                           setChoices((choices) => {
                             return {
                               ...choices,
-                              [opt.displayName]: v.label,
+                              [opt.displayName.toLowerCase()]: v.label.toLowerCase(),
                             }
                           })
                         }}
@@ -142,6 +146,7 @@ const ProductView: FC<Props> = ({ product }) => {
               className={s.button}
               onClick={addToCart}
               loading={loading}
+              disabled={!variant}
             >
               Add to Cart
             </Button>

components/product/helpers.ts

@@ -1,5 +1,3 @@
-import type { ProductNode } from '@framework/api/operations/get-product'
-
 export type SelectedOptions = {
   size: string | null
   color: string | null
@@ -10,42 +8,18 @@ export type ProductOption = {
   values: any
 }
 
-// Returns the available options of a product
-export function getProductOptions(product: ProductNode) {
-  const options = product.productOptions.edges?.reduce<ProductOption[]>(
-    (arr, edge) => {
-      if (edge?.node.__typename === 'MultipleChoiceOption') {
-        arr.push({
-          displayName: edge.node.displayName.toLowerCase(),
-          values: edge.node.values.edges?.map((edge) => edge?.node),
-        })
-      }
-      return arr
-    },
-    []
-  )
-
-  return options
-}
-
-// Finds a variant in the product that matches the selected options
-export function getCurrentVariant(product: ProductNode, opts: SelectedOptions) {
-  const variant = product.variants.edges?.find((edge) => {
-    const { node } = edge ?? {}
-
+export function getVariant(product: Product, opts: SelectedOptions) {
+  const variant = product.variants.find((variant) => {
     return Object.entries(opts).every(([key, value]) =>
-      node?.productOptions.edges?.find((edge) => {
+      variant.options.find((option) => {
         if (
-          edge?.node.__typename === 'MultipleChoiceOption' &&
-          edge.node.displayName.toLowerCase() === key
+          option.__typename === 'MultipleChoiceOption' &&
+          option.displayName.toLowerCase() === key.toLowerCase()
         ) {
-          return edge.node.values.edges?.find(
-            (valueEdge) => valueEdge?.node.label === value
-          )
+          return option.values.find((v) => v.label.toLowerCase() === value)
         }
       })
     )
   })
-
   return variant
 }

components/wishlist/WishlistCard/WishlistCard.tsx

@@ -2,21 +2,20 @@ import { FC, useState } from 'react'
 import cn from 'classnames'
 import Link from 'next/link'
 import Image from 'next/image'
-import type { WishlistItem } from '@framework/api/wishlist'
-import usePrice from '@framework/product/use-price'
-import useRemoveItem from '@framework/wishlist/use-remove-item'
-import useAddItem from '@framework/cart/use-add-item'
-import { useUI } from '@components/ui/context'
-import { Button, Text } from '@components/ui'
-import { Trash } from '@components/icons'
 import s from './WishlistCard.module.css'
+import { Trash } from '@components/icons'
+import { Button, Text } from '@components/ui'
+
+import { useUI } from '@components/ui/context'
+import usePrice from '@framework/product/use-price'
+import useAddItem from '@framework/cart/use-add-item'
+import useRemoveItem from '@framework/wishlist/use-remove-item'
 
 interface Props {
-  item: WishlistItem
+  product: Product
 }
 
-const WishlistCard: FC<Props> = ({ item }) => {
-  const product = item.product!
+const WishlistCard: FC<Props> = ({ product }) => {
   const { price } = usePrice({
     amount: product.prices?.price?.value,
     baseAmount: product.prices?.retailPrice?.value,
@@ -34,7 +33,7 @@ const WishlistCard: FC<Props> = ({ item }) => {
     try {
       // If this action succeeds then there's no need to do `setRemoving(true)`
       // because the component will be removed from the view
-      await removeItem({ id: item.id! })
+      await removeItem({ id: product.id! })
     } catch (error) {
       setRemoving(false)
     }
@@ -43,8 +42,8 @@ const WishlistCard: FC<Props> = ({ item }) => {
     setLoading(true)
     try {
       await addItem({
-        productId: product.entityId,
-        variantId: product.variants.edges?.[0]?.node.entityId!,
+        productId: Number(product.id),
+        variantId: Number(product.variants[0].id),
       })
       openSidebar()
       setLoading(false)
@@ -57,10 +56,10 @@ const WishlistCard: FC<Props> = ({ item }) => {
     <div className={cn(s.root, { 'opacity-75 pointer-events-none': removing })}>
       <div className={`col-span-3 ${s.productBg}`}>
         <Image
-          src={product.images.edges?.[0]?.node.urlOriginal!}
+          src={product.images[0].url}
           width={400}
           height={400}
-          alt={product.images.edges?.[0]?.node.altText || 'Product Image'}
+          alt={product.images[0].alt || 'Product Image'}
         />
       </div>
 

framework/bigcommerce/api/catalog/products.ts

@@ -21,7 +21,7 @@ export type ProductsHandlers = {
 
 const METHODS = ['GET']
 
-// TODO: a complete implementation should have schema validation for `req.body`
+// TODO(lf): a complete implementation should have schema validation for `req.body`
 const productsApi: BigcommerceApiHandler<
   SearchProductsData,
   ProductsHandlers

framework/bigcommerce/lib/normalize.ts

@@ -1,5 +1,19 @@
 import { Product as BCProduct } from '@framework/schema'
 
+function productOptionNormalize({
+  node: {
+    entityId,
+    values: { edges },
+    ...rest
+  },
+}: any) {
+  return {
+    id: entityId,
+    values: edges.map(({ node }: any) => node),
+    ...rest,
+  }
+}
+
 export function normalizeProduct(productNode: BCProduct): Product {
   const {
     entityId: id,
@@ -10,6 +24,7 @@ export function normalizeProduct(productNode: BCProduct): Product {
     path,
     id: _,
     options: _0,
+    brand,
     ...rest
   } = productNode
 
@@ -27,26 +42,21 @@ export function normalizeProduct(productNode: BCProduct): Product {
         )
       : [],
     variants: variants.edges
-      ? variants.edges.map(({ node: { entityId, ...rest } }: any) => ({
-          id: entityId,
-          ...rest,
-        }))
-      : [],
-    options: productOptions.edges
-      ? productOptions.edges.map(
-          ({
-            node: {
-              entityId,
-              values: { edges },
-              ...rest
-            },
-          }: any) => ({
+      ? variants.edges.map(
+          ({ node: { entityId, productOptions, ...rest } }: any) => ({
             id: entityId,
-            values: edges.map(({ node }: any) => node),
+            options: productOptions.edges.map(productOptionNormalize),
             ...rest,
           })
         )
       : [],
+    options: productOptions.edges
+      ? productOptions.edges.map(productOptionNormalize)
+      : [],
+    brand: {
+      id: brand?.entityId,
+      ...brand,
+    },
     price: {
       value: prices?.price.value,
       currencyCode: prices?.price.currencyCode,

framework/bigcommerce/wishlist/use-wishlist.tsx

@@ -46,7 +46,7 @@ export function extendHook(
     const response = useCommerceWishlist(
       defaultOpts,
       [
-        ['customerId', customer?.entityId],
+        ['customerId', customer?.id],
         ['includeProducts', includeProducts],
       ],
       customFetcher,

framework/types.d.ts

@@ -31,6 +31,7 @@ interface ProductImage {
 
 interface ProductVariant {
   id: string | number
+  options: ProductOption[]
 }
 
 interface ProductPrice {
@@ -51,21 +52,17 @@ interface Wishlist extends Entity {
 
 interface Order {}
 
-interface Customer extends Entity {
-  [prop: string]: any
-}
+interface Customer extends Entity {}
 
 type UseCustomerResponse = {
   customer: Customer
 } | null
 
 interface Category extends Entity {
-  id: string
   name: string
 }
 
 interface Brand extends Entity {
-  id: string
   name: string
 }