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

fix: cover case when all material needs to be bought #29326

Merged
merged 1 commit into from
Jan 17, 2022

Conversation

ankush
Copy link
Member

@ankush ankush commented Jan 17, 2022

When a material request is to be made for purchase qty should be converted to purchase UOM.

Identified missing case when reviewing #28570 (comment)

When material request is to be made for purchase qty should be converted
to purchase UOM
@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Jan 17, 2022
@ankush ankush added backport version-13-hotfix and removed needs-tests This PR needs automated unit-tests. labels Jan 17, 2022
@ankush ankush changed the title fix: conver case when all material needs to be bought fix: cover case when all material needs to be bought Jan 17, 2022
@ankush ankush added the Skip Manual Testing The changes in this PR does not require manual testing label Jan 17, 2022
@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #29326 (32ea18d) into develop (9cd26fb) will increase coverage by 0.12%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #29326      +/-   ##
===========================================
+ Coverage    57.87%   58.00%   +0.12%     
===========================================
  Files         1110     1110              
  Lines        68094    68091       -3     
===========================================
+ Hits         39412    39495      +83     
+ Misses       28682    28596      -86     
Impacted Files Coverage Δ
...cturing/doctype/production_plan/production_plan.py 69.18% <ø> (+0.39%) ⬆️
...t/product_bundle_balance/product_bundle_balance.py 79.54% <0.00%> (-2.28%) ⬇️
...r/doctype/leave_ledger_entry/leave_ledger_entry.py 90.16% <0.00%> (-1.64%) ⬇️
erpnext/portal/utils.py 28.98% <0.00%> (-1.45%) ⬇️
...t/accounts/report/general_ledger/general_ledger.py 59.93% <0.00%> (-1.35%) ⬇️
...cial_statement/consolidated_financial_statement.py 89.77% <0.00%> (-0.64%) ⬇️
...rpnext/stock/report/stock_balance/stock_balance.py 79.16% <0.00%> (-0.60%) ⬇️
erpnext/stock/stock_ledger.py 85.42% <0.00%> (-0.56%) ⬇️
erpnext/accounts/utils.py 59.09% <0.00%> (+0.18%) ⬆️
...stock/doctype/purchase_receipt/purchase_receipt.py 90.69% <0.00%> (+0.26%) ⬆️
... and 13 more

@ankush ankush merged commit 48f3d53 into frappe:develop Jan 17, 2022
mergify bot pushed a commit that referenced this pull request Jan 17, 2022
When material request is to be made for purchase qty should be converted
to purchase UOM

(cherry picked from commit 48f3d53)
@ankush ankush deleted the all_bought branch January 17, 2022 16:45
ankush added a commit that referenced this pull request Jan 17, 2022
When material request is to be made for purchase qty should be converted
to purchase UOM

(cherry picked from commit 48f3d53)

Co-authored-by: Ankush Menat <ankush@frappe.io>
@ankush ankush removed the Skip Manual Testing The changes in this PR does not require manual testing label Mar 9, 2022
KrithiRamani pushed a commit to KrithiRamani/erpnext that referenced this pull request Mar 14, 2022
…rappe#29327)

When material request is to be made for purchase qty should be converted
to purchase UOM

(cherry picked from commit 48f3d53)

Co-authored-by: Ankush Menat <ankush@frappe.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant