-
Notifications
You must be signed in to change notification settings - Fork 15
Improve UX clarity between "Sync Now" and "Import all Products from Square" buttons #445
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
…ration. Updated descriptions to clarify the distinction between importing new products and syncing existing ones. Added informational notices in the admin settings and sync pages to guide users effectively.
|
This is linked to GitHub issue #446. |
|
This is linked to GitHub issue #447. |
Removed redundant notice about importing new products from Square in the Settings Page. Updated the Sync Page to conditionally display the description for syncing existing products, enhancing clarity for users.
…otice. Updated header and conditional display of sync information to enhance user understanding of product syncing with Square.
Update product import and sync messaging for clarity in WooCommerce Square integration. Enhanced descriptions in the Settings and Sync pages to differentiate between importing new products and syncing existing ones based on the system of record. Improved user guidance for product management.
|
Thanks for the kind and detailed feedback, @iamdharmesh. Should look okay now. |
iamdharmesh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
@iamdharmesh updated PR description accordingly. Thanks! |
qasumitbagthariya
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA Update ✅
I have verified this PR in the fix/square-225 branch, which has been fixed and is functioning as intended.
I tested the following on this branch:
- The "Sync Now" button now shows helper text: "Updates existing synced products only.
- Sync Now" and verify the modal includes a yellow warning note about scope
- Import all Products from Square" button has description text below it
- Setup Wizard
Testing Environment
Details
- WordPress: 6.9
- Theme: Twenty Twenty-Five 1.4
- WooCommerce - 10.4.0
- PHP: 8.0.30
- Web Server: Nginx 1.20.2
- Browser: Chrome
- OS: macOS 15.2
- Branch: fix/square-225
Steps to Test- As mentioned in the PR description.
Test Results - It is working as expected.
Functional Demo / Screencast -
Special Notes - Ready for UAT
Testing Document status:
Cases related to this Issue/PR are added to the Critical Flow Wiki pages:
- Yes
- Not Required/Applicable for this PR
All Submissions:
Changes proposed in this Pull Request:
This PR improves clarity between the "Sync Now" and "Import all Products from Square" actions to reduce merchant confusion.
Problem: Merchants were confused because both buttons lead to similar-looking pages, but they do different things:
This caused support cases where merchants couldn't understand why new products added in Square weren't appearing after using "Sync Now".
Solution: Added descriptive helper text in key locations to clarify the difference:
Update tab: Added helper text below the "Sync Now" button and a warning notice in the sync modal explaining it only updates existing synced products (when Square is the system of record)
Settings tab (PHP): Added description text below the "Import all Products from Square" button explaining how it differs from "Sync Now"
Settings tab (React): Added the same description text and localized the import modal title
Closes #447
Closes https://linear.app/a8c/issue/SQUARE-225/improve-ux-clarity-between-sync-now-and-import-all-products-from.
Steps to test the changes in this Pull Request:
Changelog entry