Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tokenized ECE not able to parse selected variation w/ "Variation Swatches for WooCommerce" #10303

Open
frosso opened this issue Feb 5, 2025 · 0 comments · May be fixed by #10306
Open

Tokenized ECE not able to parse selected variation w/ "Variation Swatches for WooCommerce" #10303

frosso opened this issue Feb 5, 2025 · 0 comments · May be fixed by #10306
Assignees
Labels
type: bug The issue is a confirmed bug.

Comments

@frosso
Copy link
Contributor

frosso commented Feb 5, 2025

Describe the bug

Internally reported here: p1738743947909059/1738707211.859139-slack-CGGCLBN58

The "Variation Swatches for WooCommerce" plugin alters the site's markup for the variations, causing the ECE to not be able to parse the selected variation on a product page.

This causes a Store API add-to-cart call made with incorrect values causing it to fail with woocommerce_rest_variation_id_from_variation_data, preventing the product from being able to check out within the scope of the ECE dialog.

Image

The product can still be added to the cart via the "Add to cart" button.

ECE on cart & checkout pages remains unaffected.

To Reproduce

  • Install the plugin
  • Ensure you have GooglePay/ApplePay enabled
  • Click on the GooglePay/ApplePay button
  • Observe the add-tocart` call result

Actual behavior

Screenshots

Expected behavior

Product pages using "Variation Swatches for WooCommerce" can still add the product to the cart.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

@frosso frosso added the type: bug The issue is a confirmed bug. label Feb 5, 2025
@frosso frosso self-assigned this Feb 6, 2025
@frosso frosso linked a pull request Feb 6, 2025 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant