Automattic / wc-calypso-bridge

20 stars 4 forks source link

Deactivate WooCommerce Services if either Woo Shipping or Woo Tax is active #1458

Closed waclawjacek closed 6 months ago

waclawjacek commented 6 months ago

Changes proposed in this Pull Request:

Closes #1457.

This PR replicates what was done in #1430 to ensure the WooCommerce Shipping & Tax extension is not running if either Woo Shipping or Woo Tax is present but only on Woo Express sites.

How to test the changes in this Pull Request:

SSH into your dev site and tail -f /tmp/php-errors - Monitor for warnings, fatals related to this PR

  1. Set up a Woo Express dev site by following the instructions in the "How to set up an effective development environment for WooExpress & free trial?" section of this page: PdibGW-Uj-p2
  2. Make sure you are not sandboxing your WPCOM payment methods.
  3. Check out this branch and sync it to your Woo Express dev site.
  4. On your test site, install the following plugins:
  5. Test different combinations of enabling the plugins.
    • Whenever WCS&T is active and another plugin is activated (either Shipping or Tax), WCS&T should get deactivated. Reasoning: p1713528855792869/1713439513.024539-slack-C04KWSNPSE5
    • Whenever Woo Shipping or Woo Tax are active and you try to activate WCS&T, it should not activate.
    • Verify the changes work on Woo Express sites as well as the Entrepreneur plan on WPCOM, but not other WPCOM plans: https://wordpress.com/start/

Other information:

FOR PR REVIEWER ONLY:

github-actions[bot] commented 6 months ago

Size Change: 0 B

Total Size: 197 kB

ℹ️ View Unchanged | Filename | Size | | :--- | :---: | | `./build/53.js` | 1.05 kB | | `./build/index.css` | 883 B | | `./build/index.js` | 122 kB | | `./build/marketing.js` | 58 kB | | `./build/payment-gateway-suggestions.css` | 1.25 kB | | `./build/payment-gateway-suggestions.js` | 6.46 kB | | `./build/plugins.js` | 3.92 kB | | `./build/style-index.css` | 2.15 kB | | `./build/style-marketing.css` | 805 B |

compressed-size-action