Conversation
…y is missing in sync and product handlers.
iamdharmesh
approved these changes
Dec 9, 2025
qasumitbagthariya
approved these changes
Dec 15, 2025
Collaborator
qasumitbagthariya
left a comment
There was a problem hiding this comment.
QA Update ✅
I have verified this PR in the fix/square-214 branch, which has been fixed and is functioning as intended.
I tested the following on this branch:
Testing Environment
Details
- WordPress: 6.9
- Theme: Storefront 4.6.2
- 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-214
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
Collaborator
Regression / Smoke Test Report ✅Tested with Archive File created via "php woorelease.phar build repo_URL" (Composer version 2.8.4, npm version 10.8.2, node version 20.18.1) Status- Working expected with Plugin Archive/Zip file same as fix specific branch. Testing Environment Details
Next Step- Ready to Merge 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All Submissions:
Changes proposed in this Pull Request:
When products are imported to Square via CSV with inventory tracking disabled, the Square API response does not include the track_inventory field. The current code defaults this missing field to true, which incorrectly enables "Manage Stock" in WooCommerce, resulting in products showing as "Out of Stock."
Products created directly in Square's dashboard work correctly because they explicitly set track_inventory: false in the API response.
This PR changes the default value from true to false when the track_inventory key is missing, ensuring CSV-imported products without inventory tracking are correctly imported as "In Stock" with stock management disabled.
Closes https://linear.app/a8c/issue/SQUARE-214/csv-imported-products-with-non-tracked-inventory-are-set-to-manage.
Steps to test the changes in this Pull Request:
Use this test.csv file if needed.
Additional verification:
Changelog entry