zaviagodev / frappe

MIT License
0 stars 1 forks source link

Issue with stock transfer in Stock Level widget in Items tab #217

Open John-zaviago opened 4 months ago

John-zaviago commented 4 months ago

In the stock transfer section, there are three buttons available: Remove, Move, and Add.

However, it has been observed that the Move button currently functions similarly to the Add button. Instead of transferring stock from one warehouse to another by removing it from the original warehouse and adding it to the target warehouse, the Move function only adds stock without deducting it from the source warehouse. This results in inaccurate stock movements and inventory management.

Image

Desired Solution: To address this issue effectively, the following steps can be taken:

  1. Modify Move Button Functionality: Enhance the Move button functionality to perform proper stock transfers by deducting stock from the source warehouse and adding it to the target warehouse.
  2. Update Stock Transfer Logic: Adjust the stock transfer logic to accurately reflect the movement of stock between warehouses and maintain correct inventory balances.

Expected Outcome: Upon implementing the necessary changes, the Move button in the stock transfer section will function as intended, facilitating accurate and efficient stock transfers between warehouses. This will ensure correct inventory management and minimize discrepancies in stock levels.

umer2001 commented 4 months ago

@John-zaviago fixed in tmp.zaviago.com.

relative commit: https://github.com/zaviagodev/zaviagocore/commit/80b6ada273ba92d66fccb0336f73e0ac621c00e0

chomchuen commented 3 months ago

Fixed.✅

chomchuen commented 3 months ago

@umer2001 Ready for moving to apps.hosting.

but before pushing to apps.hosting, please make sure this issue has been resolved -> https://github.com/zaviagodev/frappe/issues/229