-
Notifications
You must be signed in to change notification settings - Fork 10
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
With Woo set as Sync - Variations are incorrectly set to manage stock and backorders disabled #262
Comments
Merchant described the issue a bit differently but I think it is the same as the issue above:
Further description:
Though I am a bit confused about the -1 value appearing after tracking is disabled on Square. |
Hi @kaushikasomaiya, Thank you for reporting the issue.
I attempted to create a product and sync it with Square, and it worked correctly on my end. "Tracking" is not being enabled on the Square side, and there are no changes during the pull inventory action. Could you please confirm if stock tracking is being enabled by Square systems like the online store (as mentioned in the comment) or if tracking is enabled since the initial sync for you? If yes could you please help with more detailed steps here? Regarding backorders, we do not update this during inventory updates, so it remains the default WooCommerce value. However, during product import (when we import products by clicking the import button), it gets updated to "Do not allow", but that is more of a Square SOR thing. @qasumitbagthariya Could you please try reproducing this issue on your end?
By "order comes through online" merchant means Square online store here? Thank you. |
Hi @iamdharmesh Thank you for looking into it
When I disable stock tracking on Square for Variation B, further things appear fine it never gets enabled again. Does this help? Merchant mentioned their simple products which don't have manage stock enabled are also getting tracked on Square and -1 is set as inventory which is then pulled back. However I think this is a different issue if at all, and might be a conflict so I've asked for more details. |
Hello @kaushikasomaiya Thanks for the detailed comment. I followed the steps you provided to reproduce the issue but was unable to replicate it. Please refer to the attached video for more details. Let me know if I missed anything or if there's anything else you'd like me to check. Screen.Recording.2025-01-06.at.4.05.25.PM.1.movcc @iamdharmesh |
Hi @iamdharmesh @qasumitbagthariya Please find the video below of both issues (262 and 261, around 1:47, you'll see issue 261) |
Thanks for providing the video, @kaushikasomaiya. I was enabling 'Sync with Square?' from the product edit screen, and it was working fine there. The issue was occurring when 'Sync with Square?' was enabled via bulk or quick edit. I’ve raised #264 to fix it. Also, I’m not able to replicate issue 261 on my side as shown in the video, but I’ll investigate further when we start looking into it. |
Hi @iamdharmesh and team, Just had a couple of questions around pull_inventory (with Woo SOR and inventory sync enabled)
I am trying to understand what should be the ideal behaviour here. |
Hi @kaushikasomaiya, Thank you for your questions. This is expected behavior or perhaps more accurately a known issue. When we sync a product with stock management disabled it disables the tracking on the Square side and whenever tracking is disabled on the Square side, the product is updated as "in stock" in WooCommerce with "manage stock" disabled. We currently have an open issue for this: #21. This issue was on hold because the
Hope this makes things clear, but please feel free to ask if you need any further information or have any questions. Thank you. |
Hi @iamdharmesh Thanks for fixing this. Just sharing the results of investigation done on this merchant's ticket - I think a import was done with 800~ variable products set to manage stock and no stock value (NULL) and this got synced to Square - this is why Pull Inventory was marking the variations to manage stock. I think regardless this fix, the issue can be replicated if a bulk import is done via default Woo importer or other tool with Stock management enabled on parent and Sync column in the CSV set to yes. Maybe it is more of something the merchants need to be aware of while importing products when Square and inventory sync are enabled. |
Hi @kaushikasomaiya, Thanks for sharing this information. We already display a notice on the product import screen regarding inventory sync. However, this is something that merchants can easily miss. I will check the best way to handle this and will update you here soon. Thanks for providing all this important information. |
8942863-zen
Let's say we have a variable product with two variations.
At global level:
Full Size: https://d.pr/i/c56IIg
This means the variations which do not have manage stock enabled are expected to follow global settings.
Keep Square plugin turned off while creating the product above, and once ready enable Square and set to sync.
The product gets synced to Square however Variation B is marked as "track" stock on Square. A pull inventory action modifies Variation B on Woo and sets stock management on it with backorders disabled.
The net result of this is despite global setting being set to allow backorders, as soon as inventory goes 0 or negative Variation B is out of stock on Woo for customers.
Sidenote: The reason I turned off Square while creating a variable product is because I feel the
wc-square-admin-products.js
is buggy and has multiple issues. For instance, strings like these appearing on checking / unchecking the sync, under inventory the Stock Quantity box visible though manage stock is unchecked, and the other issue I reported here #261. I feel the script needs to be thoroughly reviewed for consistency.The text was updated successfully, but these errors were encountered: