Automattic / newspack-theme

A theme for Newspack.
https://newspack.com
GNU General Public License v2.0
305 stars 66 forks source link

fix: update template for woo 8.7.0 #2264

Closed laurelfulford closed 7 months ago

laurelfulford commented 7 months ago

All Submissions:

Changes proposed in this Pull Request:

Replaces #2263; I switched it to a hotfix since these alerts can cause worries for publishers/lot of communication for support.

This PR gets the order-details-customer.php file in the Newspack theme up-to-date with what was changed in the template in the WooCommerce 8.7.0 release, to prevent a message from displaying.

See 1200550061930446-as-1206883834710822

How to test the changes in this Pull Request:

  1. Make sure WooCommerce is up-to-date on your test site.
  2. View the WP Admin; note you have a notice like this:

image

  1. Apply this PR.
  2. Confirm that the notice goes away.
  3. Navigate to /my-account > Orders, and confirm there are no issues.
  4. Spot-check the file's changes against the ones made in the WooCommerce plugin here, to make sure they line up.

Other information:

matticbot commented 7 months ago

:tada: This PR is included in version 1.87.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: