Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Fix undo button #9478

Merged
merged 3 commits into from
May 17, 2023
Merged

Fix undo button #9478

merged 3 commits into from
May 17, 2023

Conversation

gigitux
Copy link
Contributor

@gigitux gigitux commented May 16, 2023

This PR fixes the undo button introduced with #9386.

Testing

User Facing Testing

  1. Open the Single Product Template.
  2. Be sure that the description is: "Transform this template into multiple blocks so you can add, remove, reorder, and customize your single product template (for other templates, this should correspond to the name of the template)."
  3. Hover the button "transform into blocks".
  4. Ensure the preview corresponds to the "blockified template".
  5. Click the button.
  6. Ensure that the group block around the blockified product is selected.
  7. Ensure the snackbar is visible and the undo button works correctly.
  • Do not include in the Testing Notes

WooCommerce Visibility

  • WooCommerce Core
  • Feature plugin
  • Experimental

@gigitux gigitux added type: bug The issue/PR concerns a confirmed bug. skip-changelog PRs that you don't want to appear in the changelog. labels May 16, 2023
@gigitux gigitux self-assigned this May 16, 2023
@woocommercebot woocommercebot requested review from a team and kmanijak and removed request for a team May 16, 2023 07:58
@github-actions
Copy link
Contributor

github-actions bot commented May 16, 2023

The release ZIP for this PR is accessible via:

https://wcblocks.wpcomstaging.com/wp-content/uploads/woocommerce-gutenberg-products-block-9478.zip

Script Dependencies Report

There is no changed script dependency between this branch and trunk.

This comment was automatically generated by the ./github/compare-assets action.

TypeScript Errors Report

  • Files with errors: 472
  • Total errors: 2269

🎉 🎉 This PR does not introduce new TS errors.

comments-aggregator

@github-actions
Copy link
Contributor

github-actions bot commented May 16, 2023

Size Change: +245 B (0%)

Total Size: 1.08 MB

Filename Size Change
build/active-filters-frontend.js 8.57 kB +3 B (0%)
build/active-filters-wrapper-frontend.js 7.62 kB -4 B (0%)
build/active-filters.js 7.48 kB +1 B (0%)
build/all-products-frontend.js 11.9 kB +7 B (0%)
build/all-products.js 39.2 kB +141 B (0%)
build/all-reviews.js 7.77 kB +1 B (0%)
build/attribute-filter-wrapper--stock-filter-wrapper-frontend.js 4.05 kB -1 B (0%)
build/attribute-filter-wrapper-frontend.js 4.28 kB -3 B (0%)
build/attribute-filter.js 13.1 kB -4 B (0%)
build/breadcrumbs.js 2.05 kB +3 B (0%)
build/cart-blocks/cart-accepted-payment-methods-frontend.js 1.39 kB -2 B (0%)
build/cart-blocks/cart-cross-sells-products--product-price-frontend.js 2.94 kB +1 B (0%)
build/cart-blocks/cart-cross-sells-products-frontend.js 3.73 kB -2 B (0%)
build/cart-blocks/cart-express-payment--checkout-blocks/express-payment-frontend.js 5.17 kB +3 B (0%)
build/cart-blocks/cart-express-payment-frontend.js 717 B -2 B (0%)
build/cart-blocks/cart-line-items--mini-cart-contents-block/products-table-frontend.js 5.48 kB +107 B (+2%)
build/cart-blocks/order-summary-shipping-frontend.js 17 kB -40 B (0%)
build/cart-blocks/order-summary-taxes-frontend.js 435 B +2 B (0%)
build/cart-frontend.js 29.7 kB -20 B (0%)
build/cart.js 45 kB +109 B (0%)
build/catalog-sorting.js 1.7 kB -3 B (0%)
build/checkout-blocks/actions-frontend.js 1.85 kB +2 B (0%)
build/checkout-blocks/billing-address--checkout-blocks/shipping-address-frontend.js 4.7 kB -29 B (-1%)
build/checkout-blocks/billing-address-frontend.js 1.18 kB -2 B (0%)
build/checkout-blocks/contact-information-frontend.js 2.05 kB -1 B (0%)
build/checkout-blocks/express-payment-frontend.js 1.13 kB -6 B (-1%)
build/checkout-blocks/order-note-frontend.js 1.14 kB -1 B (0%)
build/checkout-blocks/order-summary-cart-items-frontend.js 3.69 kB -1 B (0%)
build/checkout-blocks/order-summary-coupon-form-frontend.js 1.79 kB -2 B (0%)
build/checkout-blocks/order-summary-discount-frontend.js 2.29 kB -5 B (0%)
build/checkout-blocks/order-summary-fee-frontend.js 276 B +1 B (0%)
build/checkout-blocks/order-summary-frontend.js 1.28 kB -2 B (0%)
build/checkout-blocks/order-summary-shipping-frontend.js 17 kB -43 B (0%)
build/checkout-blocks/order-summary-subtotal-frontend.js 273 B -1 B (0%)
build/checkout-blocks/order-summary-taxes-frontend.js 435 B +2 B (0%)
build/checkout-blocks/payment-frontend.js 8.29 kB +21 B (0%)
build/checkout-blocks/shipping-address-frontend.js 1.17 kB -3 B (0%)
build/checkout-blocks/shipping-method-frontend.js 2.59 kB -1 B (0%)
build/checkout-blocks/shipping-methods-frontend.js 6.36 kB +13 B (0%)
build/checkout.js 46.3 kB -7 B (0%)
build/featured-category.js 15 kB -3 B (0%)
build/featured-product.js 15.2 kB -7 B (0%)
build/filter-wrapper-frontend.js 14.2 kB +39 B (0%)
build/filter-wrapper.js 2.39 kB +3 B (0%)
build/handpicked-products.js 8 kB +1 B (0%)
build/legacy-template.js 6.45 kB +121 B (+2%)
build/mini-cart-component-frontend.js 28.4 kB -544 B (-2%)
build/mini-cart-contents-block/cart-button-frontend.js 1.73 kB -235 B (-12%) 👏
build/mini-cart-contents-block/checkout-button-frontend.js 1.74 kB -233 B (-12%) 👏
build/mini-cart-contents-block/footer-frontend.js 4.1 kB -200 B (-5%)
build/mini-cart-contents-block/products-table-frontend.js 591 B -1 B (0%)
build/mini-cart-contents-block/shopping-button-frontend.js 528 B -229 B (-30%) 🎉
build/mini-cart-contents-block/title-label-frontend.js 1.54 kB +1 B (0%)
build/mini-cart-contents.js 18 kB +94 B (+1%)
build/mini-cart-frontend.js 2.15 kB +1 B (0%)
build/mini-cart.js 4.2 kB -1 B (0%)
build/price-filter-wrapper-frontend.js 6.75 kB +3 B (0%)
build/price-filter.js 8.47 kB -2 B (0%)
build/product-add-to-cart-frontend.js 6.52 kB +6 B (0%)
build/product-add-to-cart.js 8.87 kB +6 B (0%)
build/product-button-frontend.js 2.65 kB -1 B (0%)
build/product-categories.js 2.37 kB -1 B (0%)
build/product-category.js 9.35 kB +1 B (0%)
build/product-collection.js 3.16 kB +162 B (+5%) 🔍
build/product-image-frontend.js 2.63 kB -1 B (0%)
build/product-image.js 4.18 kB -1 B (0%)
build/product-new.js 8.34 kB -6 B (0%)
build/product-on-sale.js 8.68 kB -1 B (0%)
build/product-query.js 11.7 kB +81 B (+1%)
build/product-rating-frontend.js 2.31 kB -4 B (0%)
build/product-results-count.js 1.66 kB -1 B (0%)
build/product-sale-badge-frontend.js 1.8 kB -1 B (0%)
build/product-search.js 2.63 kB +1 B (0%)
build/product-sku-frontend.js 1.84 kB -3 B (0%)
build/product-stock-indicator-frontend.js 2.02 kB -2 B (0%)
build/product-stock-indicator.js 731 B +1 B (0%)
build/product-summary-frontend.js 2.19 kB -1 B (0%)
build/product-tag.js 8.97 kB -1 B (0%)
build/product-template.js 3.26 kB -15 B (0%)
build/product-title-frontend.js 2.22 kB -1 B (0%)
build/product-title.js 3.69 kB -3 B (0%)
build/products-by-attribute.js 9.7 kB -1 B (0%)
build/rating-filter-frontend.js 21.3 kB -26 B (0%)
build/rating-filter-wrapper-frontend.js 6.21 kB -2 B (0%)
build/rating-filter.js 6.88 kB -6 B (0%)
build/reviews-by-category.js 12.1 kB -6 B (0%)
build/reviews-frontend.js 7.11 kB +7 B (0%)
build/single-product.js 11.1 kB -2 B (0%)
build/stock-filter.js 7.6 kB -3 B (0%)
build/store-notices.js 1.69 kB -2 B (0%)
build/vendors--attribute-filter-wrapper--cart-blocks/order-summary-coupon-form--cart-blocks/order-summary--48e1e4bb-frontend.js 6.82 kB -1 B (0%)
build/vendors--attribute-filter-wrapper--cart-blocks/order-summary-shipping--checkout-blocks/billing-addr--d9f38f9d-frontend.js 4.21 kB +1 B (0%)
build/vendors--attribute-filter-wrapper--stock-filter-wrapper-frontend.js 5.11 kB +2 B (0%)
build/vendors--cart-blocks/order-summary-shipping--checkout-blocks/billing-address--checkout-blocks/order--decc3dc6-frontend.js 19.4 kB -1 B (0%)
build/vendors--checkout-blocks/shipping-method-frontend.js 12.5 kB -1 B (0%)
build/vendors--product-add-to-cart-frontend.js 7.26 kB -3 B (0%)
build/vendors--rating-filter-wrapper-frontend.js 5.11 kB +3 B (0%)
build/wc-blocks-style-rtl.css 27.8 kB -26 B (0%)
build/wc-blocks-style.css 27.8 kB -29 B (0%)
build/wc-blocks-vendors.js 65.1 kB +11 B (0%)
build/wc-blocks.js 3.7 kB +1.07 kB (+41%) 🚨
ℹ️ View Unchanged
Filename Size
build/blocks-checkout.js 35.1 kB
build/cart-blocks/cart-cross-sells-frontend.js 253 B
build/cart-blocks/cart-items-frontend.js 301 B
build/cart-blocks/cart-line-items-frontend.js 1.06 kB
build/cart-blocks/cart-order-summary-frontend.js 1.28 kB
build/cart-blocks/cart-totals-frontend.js 308 B
build/cart-blocks/empty-cart-frontend.js 345 B
build/cart-blocks/filled-cart-frontend.js 656 B
build/cart-blocks/order-summary-coupon-form-frontend.js 1.63 kB
build/cart-blocks/order-summary-discount-frontend.js 2.12 kB
build/cart-blocks/order-summary-fee-frontend.js 273 B
build/cart-blocks/order-summary-heading-frontend.js 333 B
build/cart-blocks/order-summary-subtotal-frontend.js 273 B
build/cart-blocks/proceed-to-checkout-frontend.js 1.38 kB
build/checkout-blocks/fields-frontend.js 330 B
build/checkout-blocks/pickup-options-frontend.js 4.8 kB
build/checkout-blocks/terms-frontend.js 1.56 kB
build/checkout-blocks/totals-frontend.js 312 B
build/checkout-frontend.js 31.3 kB
build/customer-account.js 3.18 kB
build/general-style-rtl.css 1.31 kB
build/general-style.css 1.31 kB
build/mini-cart-contents-block/empty-cart-frontend.js 360 B
build/mini-cart-contents-block/filled-cart-frontend.js 266 B
build/mini-cart-contents-block/items-frontend.js 237 B
build/mini-cart-contents-block/title-frontend.js 1.91 kB
build/mini-cart-contents-block/title-items-counter-frontend.js 1.61 kB
build/price-format.js 1.19 kB
build/product-add-to-cart--product-button--product-image--product-price--product-rating--product-sale-bad--49d3ecb2.js 251 B
build/product-add-to-cart--product-button--product-image--product-rating--product-title.js 151 B
build/product-best-sellers.js 8.34 kB
build/product-button--product-image--product-price--product-rating--product-sale-badge--product-sku--prod--5bce0384.js 963 B
build/product-button.js 4.01 kB
build/product-price-frontend.js 203 B
build/product-price.js 1.68 kB
build/product-rating.js 999 B
build/product-sale-badge.js 666 B
build/product-sku.js 535 B
build/product-summary.js 904 B
build/product-top-rated.js 8.58 kB
build/reviews-by-product.js 13.2 kB
build/stock-filter-wrapper-frontend.js 2.98 kB
build/vendors--cart-blocks/cart-cross-sells-products--cart-blocks/cart-line-items--cart-blocks/cart-order--3c5fe802-frontend.js 5.26 kB
build/vendors--cart-blocks/cart-line-items--checkout-blocks/order-summary-cart-items--mini-cart-contents---233ab542-frontend.js 3.57 kB
build/vendors--checkout-blocks/pickup-options--checkout-blocks/shipping-methods-frontend.js 8.25 kB
build/vendors--price-filter-wrapper-frontend.js 2.2 kB
build/wc-blocks-data.js 22.5 kB
build/wc-blocks-editor-style-rtl.css 5.96 kB
build/wc-blocks-editor-style.css 5.96 kB
build/wc-blocks-google-analytics.js 1.56 kB
build/wc-blocks-middleware.js 933 B
build/wc-blocks-registry.js 3.15 kB
build/wc-blocks-shared-context.js 1.52 kB
build/wc-blocks-shared-hocs.js 1.75 kB
build/wc-blocks-vendors-style-rtl.css 1.96 kB
build/wc-blocks-vendors-style.css 1.96 kB
build/wc-payment-method-bacs.js 816 B
build/wc-payment-method-cheque.js 811 B
build/wc-payment-method-cod.js 909 B
build/wc-payment-method-paypal.js 837 B
build/wc-settings.js 2.6 kB
build/wc-shipping-method-pickup-location.js 30.3 kB
build/woo-directives-runtime.js 2.73 kB
build/woo-directives-vendors.js 7.91 kB

compressed-size-action

Copy link
Contributor

@danielwrobert danielwrobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gigitux the undo button now works as expected, thanks for the quick fix! 🎉

One minor thing I noticed while testing: the text for the description of the Classic Product Template reads:

Transform this template into multiple blocks so you can add, remove, reorder, and customize your single product.

That seems correct but I wanted to confirm because in your testing instructions you end that sentence with single product template, not just single product.

Copy link
Contributor

@danielwrobert danielwrobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gigitux the undo button now works as expected, thanks for the quick fix! 🎉

One minor thing I noticed while testing: the text for the description of the Classic Product Template reads:

Transform this template into multiple blocks so you can add, remove, reorder, and customize your single product.

That seems correct but I wanted to confirm because in your testing instructions you end that sentence with single product template, not just single product.

@danielwrobert
Copy link
Contributor

^^ Strange that it posted my comment 2x. 🤔 🤷‍♂️

Copy link
Contributor

@danielwrobert danielwrobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll approve this under the assumption that the text in the template is correct.

If not, it seems worth adjusting quickly before merging! 😄

@github-actions github-actions bot added this to the 10.3.0 milestone May 16, 2023
@gigitux
Copy link
Contributor Author

gigitux commented May 17, 2023

I'll approve this under the assumption that the text in the template is correct.

If not, it seems worth adjusting quickly before merging! 😄

Thanks for catching this! It was a bug. We are using template at the end for other templates:

Fixed with 33fd3a0

@gigitux gigitux enabled auto-merge (squash) May 17, 2023 09:36
@gigitux gigitux merged commit 79c9459 into trunk May 17, 2023
@gigitux gigitux deleted the fix/undo-button branch May 17, 2023 13:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
skip-changelog PRs that you don't want to appear in the changelog. type: bug The issue/PR concerns a confirmed bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants