MIDNIGHT/frappe-lms
2
0
Fork 0
mirror of https://github.com/frappe/lms.git synced 2026-04-19 22:52:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2315 from raizasafeel/fix/types

Browse Source 
fix: allow None for optional args in get_order_summary
This commit is contained in:
Raizaaa
2026-04-15 10:16:11 +03:00
committed by GitHub
parent 70a9ee1bd7 193374df47
commit 0fe8d3ac74
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lms/lms/utils.py
View File

@@ -1721,7 +1721,7 @@ def get_discussion_replies(topic: str):
@frappe.whitelist()
def get_order_summary(doctype: str, docname: str, coupon: str = None, country: str = None):
def get_order_summary(doctype: str, docname: str, coupon: str | None = None, country: str | None = None):
details = get_paid_course_details(docname) if doctype == "LMS Course" else get_paid_batch_details(docname)
details.amount, details.currency = check_multicurrency(