fix: using index avoid uniqueid (#312)

* fix: using index avoid uniqueid * docs: add changelog * Update CHANGELOG.md * Update CHANGELOG.md Co-authored-by: Larícia Mota <[email protected]> Co-authored-by: Larícia Mota <[email protected]>
vtex-apps · Jul 28, 2022 · 72476e4 · 72476e4
1 parent a5662c9
commit 72476e4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+### Fixed
+- Quantity update by using item index instead of uniqueid
+
 ## [2.63.4] - 2022-05-24
 
 ### Fixed

diff --git a/react/ProductList.tsx b/react/ProductList.tsx
@@ -27,7 +27,7 @@ const ProductList: FC<Props> = ({ renderAsChildren, classes }) => {
   const { handles } = useCssHandles(CSS_HANDLES, { classes })
 
   const handleQuantityChange = useCallback(
-    (uniqueId: string, quantity: number, item: OrderFormItem) => {
+    (_: string, quantity: number, item: OrderFormItemWithIndex) => {
       if (quantity === item.quantity) {
         return
       }
@@ -56,7 +56,7 @@ const ProductList: FC<Props> = ({ renderAsChildren, classes }) => {
         })
       }
 
-      updateQuantity({ uniqueId, quantity }, options)
+      updateQuantity({ index: item.index, quantity }, options)
     },
     [push, updateQuantity]
   )

diff --git a/react/typings/OrderForm.ts b/react/typings/OrderForm.ts
@@ -26,6 +26,10 @@ interface OrderFormItem {
   parentItemIndex: number | null
 }
 
+interface OrderFormItemWithIndex extends OrderFormItem {
+  index: number
+}
+
 interface ItemAdditionalInfo {
   brandName: string
 }