Skip to content
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

feat: UOM specific barcodes #30988

Merged
merged 1 commit into from
Jun 1, 2022
Merged

Conversation

ankush
Copy link
Member

@ankush ankush commented May 12, 2022

Often warehouses might have different packaging items they stock. Like single unit, box of 100, a special pack of 42 etc.

These different packaging can be represented by UOM and people use different barcodes for each type of UOM. The problem is, when these barcodes are scanned in transaction the default UOM is fetched instead of fetching barcode-specific UOM.

This feature adds a new field on the barcode table where you can specify the UOM for any barcode. When that barcode is scanned the UOM is auto-updated on the table.

You can also scan two different UOMs in same transactions and they'll be added as separate rows instead.

Demo:

uom_scan

docs: https://docs.erpnext.com/docs/v14/user/manual/en/stock/articles/track-items-using-barcode

closes #30777

integration test: frappe/erpnext_ui_tests#112

Credit: This is largely based on proposed implementation by @szufisher ( https://discuss.erpnext.com/t/multiple-barcodes-for-different-uoms/31577/10?u=ankush )

@github-actions github-actions bot added needs-tests This PR needs automated unit-tests. stock labels May 12, 2022
@ankush ankush removed the needs-tests This PR needs automated unit-tests. label Jun 1, 2022
@ankush ankush marked this pull request as ready for review June 1, 2022 10:41
@ankush ankush force-pushed the uom_specific_barcodes branch from 4909130 to 3536603 Compare June 1, 2022 10:41
@codecov
Copy link

codecov bot commented Jun 1, 2022

Codecov Report

Merging #30988 (720e851) into develop (536f1df) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #30988      +/-   ##
===========================================
+ Coverage    63.39%   63.43%   +0.04%     
===========================================
  Files          986      986              
  Lines        67368    67368              
===========================================
+ Hits         42706    42734      +28     
+ Misses       24662    24634      -28     
Impacted Files Coverage Δ
erpnext/stock/utils.py 77.90% <ø> (ø)
...t/product_bundle_balance/product_bundle_balance.py 79.54% <0.00%> (-15.91%) ⬇️
...saction/incorrect_balance_qty_after_transaction.py 88.37% <0.00%> (-9.31%) ⬇️
.../report/bom_operations_time/bom_operations_time.py 89.13% <0.00%> (-2.18%) ⬇️
erpnext/stock/reorder_item.py 73.33% <0.00%> (-1.67%) ⬇️
erpnext/stock/doctype/warehouse/warehouse.py 77.62% <0.00%> (-1.40%) ⬇️
erpnext/stock/report/stock_ageing/stock_ageing.py 95.19% <0.00%> (-0.49%) ⬇️
...xt/e_commerce/doctype/website_item/website_item.py 47.52% <0.00%> (-0.42%) ⬇️
erpnext/controllers/stock_controller.py 91.40% <0.00%> (-0.25%) ⬇️
...xt/accounts/doctype/sales_invoice/sales_invoice.py 78.30% <0.00%> (-0.09%) ⬇️
... and 15 more

@ankush ankush merged commit 3974fbb into frappe:develop Jun 1, 2022
@ankush ankush deleted the uom_specific_barcodes branch June 1, 2022 13:08
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
Development

Successfully merging this pull request may close these issues.

UOM specific barcodes
1 participant