Merge pull request #2159 from graphcommerce-org/fix/product-aggregations

fix: prevent agragations from being excluded during defaultConfigurableOptionsSelection
graphcommerce-org · Jan 4, 2024 · 4cc7abf · 4cc7abf · vercel · Jan 4, 2024
2 parents 8421496 + 17a5304
commit 4cc7abf
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.changeset/spotty-pens-explain.md b/.changeset/spotty-pens-explain.md
@@ -0,0 +1,5 @@
+---
+"@graphcommerce/magento-product-configurable": patch
+---
+
+Spread product back into products instead of only adding items
diff --git a/packages/magento-product-configurable/utils/defaultConfigurableOptionsSelection.ts b/packages/magento-product-configurable/utils/defaultConfigurableOptionsSelection.ts
@@ -29,7 +29,7 @@ export function defaultConfigurableOptionsSelection<Q extends BaseQuery = BaseQu
 ): Q & Pick<AddProductsToCartFormProps, 'defaultValues'> {
   if (!import.meta.graphCommerce.configurableVariantForSimple) {
     const product = query?.products?.items?.find((p) => p?.url_key === urlKey)
-    return { ...query, products: { items: [product] }, defaultValues: {} }
+    return { ...query, products: { ...query?.products, items: [product] }, defaultValues: {} }
   }
 
   const simple = query?.products?.items?.find((p) => p?.url_key === urlKey)
@@ -44,7 +44,8 @@ export function defaultConfigurableOptionsSelection<Q extends BaseQuery = BaseQu
     ?.attributes
 
   const selectedOptions = (attributes ?? []).filter(nonNullable).map((a) => a.uid)
-  if (!selectedOptions.length) return { ...query, products: { items: [simple] }, defaultValues: {} }
+  if (!selectedOptions.length)
+    return { ...query, products: { ...query?.products, items: [simple] }, defaultValues: {} }
 
   /**
    * We're using writeQuery to the Apollo Client cache, to to avoid a second request to the GraphQL