Merge pull request #17084 from phot0n/fix-assigned-to-count-order

fix: use desc ordering when fetching Assigned To data
This commit is contained in:
Ankush Menat 2022-06-06 23:13:29 +05:30 committed by GitHub
commit df3601b67b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -3,6 +3,7 @@
from typing import Dict, List
import frappe
from frappe.query_builder import Order
from frappe.query_builder.functions import Count
from frappe.query_builder.terms import SubQuery
from frappe.query_builder.utils import DocType
@ -49,7 +50,7 @@ def get_group_by_count(doctype: str, current_filters: str, field: str) -> List[D
& (ToDo.reference_name.isin(SubQuery(filtered_records)))
)
.groupby(ToDo.allocated_to)
.orderby(count)
.orderby(count, order=Order.desc)
.limit(50)
.run(as_dict=True)
)