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: fixes query builder order by issue in postgres #32441

Merged
merged 3 commits into from Oct 3, 2022

Conversation

shridarpatil
Copy link
Contributor

fixed query builder order by issue in Postgres

@codecov
Copy link

codecov bot commented Oct 1, 2022

Codecov Report

Merging #32441 (0e76621) into develop (f04ae7a) will decrease coverage by 0.19%.
The diff coverage is 88.23%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #32441      +/-   ##
===========================================
- Coverage    63.90%   63.71%   -0.20%     
===========================================
  Files          817      817              
  Lines        58178    58167      -11     
===========================================
- Hits         37181    37062     -119     
- Misses       20997    21105     +108     
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%) ⬇️
.../accounts/doctype/bank_clearance/bank_clearance.py 66.66% <100.00%> (ø)
...x_withholding_category/tax_withholding_category.py 85.18% <100.00%> (+0.13%) ⬆️
...t/accounts/report/sales_register/sales_register.py 73.68% <100.00%> (-8.19%) ⬇️
erpnext/accounts/utils.py 72.80% <100.00%> (ø)
erpnext/stock/doctype/item/item.py 85.45% <100.00%> (+0.09%) ⬆️
...eport/item_variant_details/item_variant_details.py 31.32% <0.00%> (-53.02%) ⬇️
... and 33 more

@ankush ankush merged commit 69efd2e into frappe:develop Oct 3, 2022
@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
mergify bot pushed a commit that referenced this pull request Oct 3, 2022
* fix: fixes query builder order by issue in postgres

* fix: linter

(cherry picked from commit 69efd2e)
ankush pushed a commit that referenced this pull request Oct 3, 2022
…#32465)

fix: fixes query builder order by issue in postgres (#32441)

* fix: fixes query builder order by issue in postgres

* fix: linter

(cherry picked from commit 69efd2e)

Co-authored-by: Shridhar Patil <shridhar.p@zerodha.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants