Merge pull request #6002 from vuestorefront/#5981/update-url-after-change-filter

fix: update url with properly child sku

Author: Fifciu

CHANGELOG.md

@@ -44,6 +44,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Error with unknown theme/index.js alias - @Fifciuu (https://github.com/vuestorefront/vue-storefront/pull/5813)
 - ESLint warnings caused by the double import - @lukaszjedrasik
 - Fix Order History Pagination - @AishwaryShrivastav / @lukaszjedrasik ([#4599](https://github.com/vuestorefront/vue-storefront/issues/4599))
+- Fix: Updating URL's params/query params with proper child SKU if options changes - @lukaszjedrasik ([#5981](https://github.com/vuestorefront/vue-storefront/issues/5981))
 
 ### Changed / Improved
 

core/modules/catalog/events.ts

@@ -40,8 +40,9 @@ export const filterChangedProduct = async (filterOption, store, router) => {
     }
   }, { root: true })
   const { configuration, selectedVariant, options, product_option } = newProductVariant
+  const routeProp = config.seo.useUrlDispatcher ? 'query' : 'params';
+
   if (config.products.setFirstVarianAsDefaultInURL && selectedVariant) {
-    const routeProp = config.seo.useUrlDispatcher ? 'params' : 'query'
     router.push({ [routeProp]: { childSku: selectedVariant.sku } })
   }
   if (selectedVariant) {
@@ -53,6 +54,17 @@ export const filterChangedProduct = async (filterOption, store, router) => {
     )
     await store.dispatch('product/setCurrent', newProductConfiguration)
     EventBus.$emit('product-after-configure', { product: newProductConfiguration, configuration: configuration, selectedVariant: selectedVariant })
+
+    if (router?.currentRoute?.[routeProp]?.childSku) {
+      router.push(
+        {
+          [routeProp]: {
+            ...router.currentRoute[routeProp],
+            childSku: selectedVariant.sku
+          }
+        }
+      )
+    }
     return selectedVariant
   } else {
     store.dispatch('notification/spawnNotification', {