Automattic / wc-calypso-bridge

20 stars 4 forks source link

Deactivate TikTok for WooCommerce if Business is active #1430

Closed PanosSynetos closed 9 months ago

PanosSynetos commented 9 months ago

Changes proposed in this Pull Request:

Closes #1373 .

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

Master Branch uploaded on dev site

  1. Make sure your site has both tiktok-for-woocommerce.zip and tiktok-for-business.1.2.3.zip
  2. Using CLI activate both plugins
  3. wp --skip-plugins plugin activate tiktok-for-business
  4. wp --skip-plugins plugin activate tiktok-for-woocommerce
  5. wp plugin list should cause a fatal error Cannot declare class Tt4b_Mapi_Class, because the name is already in use

Checkout and upload this branch on dev site

  1. wp plugin list should not return a fatal error
  2. tiktok-for-woocommerce should be deactivated
  3. Try breaking it (activate plugins in different order, think out of the box)

Other information:

FOR PR REVIEWER ONLY:

github-actions[bot] commented 9 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