woocommerce / woocommerce-square

Square POS and Payments Integration
https://woo.com/products/square
Other
11 stars 7 forks source link

fix/165: check if dependencies are loaded before showing onboarding process #167

Closed Sidsector9 closed 4 months ago

Sidsector9 commented 4 months ago

All Submissions:


Changes proposed in this Pull Request:

Closes #165

This PR does the following if the dependencies are unmet:

  1. Does not display the onboarding wizard.
  2. Prevents direct access to the onboarding wizard via URL.
  3. Disables the "Connect to Square" button on the Settings page.
  4. Does not show the "Setup Wizard" link on the plugins page.

Steps to test the changes in this Pull Request:

Changelog entry

Fix - Check if dependencies are loaded before showing onboarding process.

qasumitbagthariya commented 4 months ago

QA Update ✅


I have checked this issue in the fix/165 branch and the related issue is resolved and working as expected.

Testing Environment

- WordPress: 6.5.5 - Theme: Storefront 4.6.0 - Theme: Twenty Twenty-Four 1.1 - WooCommerce - 9.1.2 - PHP: 8.0.30 - Web Server: Nginx 1.20.2 - Browser: Chrome - OS: macOS Ventura 13.3

Before Fix

image

After Fix

image image image image

Steps to Test- As mentioned in the PR description. Test Results - It is working as expected. Functional Demo / Screencast - Special Notes - Ready for code review (Woo) Testing Document status: Cases related to this Issue/PR are added to the Critical Flow Wiki pages:

vikrampm1 commented 4 months ago

@james-allan this PR is ready for your review.

qasumitbagthariya commented 4 months ago

Regression / Smoke Test Report ✅

Tested with Archive File created via "php woorelease.phar build repo_URL" (Composer version 2.5.5, npm version 10.7.0, node version 20.15.1)

Status- Working expected with Plugin Archive/Zip file same as fix specific branch.

Testing Environment

- WordPress: 6.6.1 - Theme: Twenty Twenty-Four 1.2 - WooCommerce - 9.1.2 - PHP: 8.0.30 - Web Server: Nginx 1.20.2 - Browser: Chrome - OS: macOS Ventura 13.3

Next Step- Ready to Merge 🚀