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: (bulk transaction) key error and better error logging #32445

Merged
merged 3 commits into from Oct 3, 2022

Conversation

rtdany10
Copy link
Contributor

@rtdany10 rtdany10 commented Oct 1, 2022

Closes #32442

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Oct 1, 2022
@codecov
Copy link

codecov bot commented Oct 1, 2022

Codecov Report

Merging #32445 (531da73) into develop (f04ae7a) will increase coverage by 0.05%.
The diff coverage is 80.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #32445      +/-   ##
===========================================
+ Coverage    63.90%   63.96%   +0.05%     
===========================================
  Files          817      817              
  Lines        58178    58161      -17     
===========================================
+ Hits         37181    37205      +24     
+ Misses       20997    20956      -41     
Impacted Files Coverage Δ
...ounts/doctype/purchase_invoice/purchase_invoice.py 83.75% <ø> (+0.27%) ⬆️
.../buying/doctype/buying_settings/buying_settings.py 100.00% <ø> (+9.09%) ⬆️
erpnext/hooks.py 100.00% <ø> (ø)
erpnext/controllers/subcontracting_controller.py 86.04% <60.00%> (-0.31%) ⬇️
erpnext/utilities/bulk_transaction.py 83.87% <66.66%> (-0.98%) ⬇️
...x_withholding_category/tax_withholding_category.py 85.18% <100.00%> (+0.13%) ⬆️
erpnext/stock/doctype/item/item.py 85.45% <100.00%> (+0.09%) ⬆️
...saction/incorrect_balance_qty_after_transaction.py 88.37% <0.00%> (-11.63%) ⬇️
...em_wise_sales_register/item_wise_sales_register.py 51.76% <0.00%> (-10.18%) ⬇️
...rpnext/accounts/doctype/shareholder/shareholder.py 80.00% <0.00%> (-10.00%) ⬇️
... and 23 more

@ankush ankush added backport version-14-hotfix backport to version 14 and removed needs-tests This PR needs automated unit-tests. labels Oct 3, 2022
@ankush ankush merged commit 9105515 into frappe:develop Oct 3, 2022
mergify bot pushed a commit that referenced this pull request Oct 3, 2022
* fix: (bulk transaction) key error and better error logging

* chore: pre-commit

* chore: linter - missing comma

(cherry picked from commit 9105515)
ankush pushed a commit that referenced this pull request Oct 3, 2022
…32445) (#32463)

fix: (bulk transaction) key error and better error logging (#32445)

* fix: (bulk transaction) key error and better error logging

* chore: pre-commit

* chore: linter - missing comma

(cherry picked from commit 9105515)

Co-authored-by: Dany Robert <danyrt@wahni.com>
frappe-pr-bot pushed a commit that referenced this pull request Oct 4, 2022
## [14.2.2](v14.2.1...v14.2.2) (2022-10-04)

### Bug Fixes

* (bulk transaction) key error and better error logging (backport [#32445](#32445)) ([#32463](#32463)) ([7a80385](7a80385))
* add non-stock item filter for `fg_item` in PO ([11f9279](11f9279))
* add validation for non-stock item in SCR ([793fa18](793fa18))
* asset requiring maintenance sold status and added test [v14-hotfix] ([c0a141b](c0a141b))
* asset tests ([6710337](6710337))
* calculate depreciation properly on asset sale entry and scrap entry ([f5f13c4](f5f13c4))
* Create accounting dimension fields in asset capitalization ([9ec0d10](9ec0d10))
* Disbursement Account in patch to update old loans ([0973989](0973989))
* don't allow to update `Maintain Stock` if the item has a `BOM` ([e5d1f59](e5d1f59))
* fetch swift_number in payment_request ([f09222a](f09222a))
* fix restore asset value after depreciation ([8605245](8605245))
* fixes query builder order by issue in postgres (backport [#32441](#32441)) ([#32465](#32465)) ([8851826](8851826))
* Incorrect TCS amount deducted in Sales Invoice ([4d51d73](4d51d73))
* Item details fetching on making transaction from item dashboard ([db60299](db60299))
* not able to return sold expired batches ([9c56a5f](9c56a5f))
* payment request make bank field Link instead of Read Only ([c8c0f34](c8c0f34))
* pick list picked-qty for batch item ([32107c4](32107c4))
* refactor asset capitilization ([bad3b33](bad3b33))
* searchfields as per customize form not working for Item ([f167abb](f167abb))
* show `Make Purchase Invoice` button based on permission ([fd86876](fd86876))
* supplied items added twice in Stock Entry ([d91ac5e](d91ac5e))
* test_scrap_asset ([fd232d1](fd232d1))
* typo in sales_register's filter mode_of_payment (backport [#32371](#32371)) ([#32446](#32446)) ([1918b5c](1918b5c))
* update with new Frappe color. fix [#32455](#32455) ([#32456](#32456)) ([3e7582f](3e7582f))

### Reverts

* Revert "fix: fetch swift_number in payment_request" ([d3c151f](d3c151f))
@barredterra barredterra mentioned this pull request Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typo in "Purchase Invoice"
2 participants