fix: make parameters of create_subscription_process optional (and other minor fixes) (#38360)

(cherry picked from commit 5a53a4b044be7e889080329056c7911970d92da8)
This commit is contained in:
Sagar Vora 2023-11-27 20:21:19 +05:30 committed by Mergify
parent 7385db0cce
commit 721b429d93
3 changed files with 3 additions and 4 deletions

View File

@ -17,11 +17,10 @@ class ProcessSubscription(Document):
def create_subscription_process(
subscription: str | None, posting_date: Union[str, datetime.date] | None
subscription: str | None = None, posting_date: Union[str, datetime.date] | None = None
):
"""Create a new Process Subscription document"""
doc = frappe.new_doc("Process Subscription")
doc.subscription = subscription
doc.posting_date = getdate(posting_date)
doc.insert(ignore_permissions=True)
doc.submit()

View File

@ -676,7 +676,7 @@ def get_prorata_factor(
def process_all(
subscription: str | None, posting_date: Optional["DateTimeLikeObject"] = None
subscription: str | None = None, posting_date: Optional["DateTimeLikeObject"] = None
) -> None:
"""
Task to updates the status of all `Subscription` apart from those that are cancelled

View File

@ -419,7 +419,6 @@ scheduler_events = {
"erpnext.projects.doctype.project.project.collect_project_status",
],
"hourly_long": [
"erpnext.accounts.doctype.process_subscription.process_subscription.create_subscription_process",
"erpnext.stock.doctype.repost_item_valuation.repost_item_valuation.repost_entries",
"erpnext.utilities.bulk_transaction.retry",
],
@ -450,6 +449,7 @@ scheduler_events = {
"erpnext.accounts.utils.auto_create_exchange_rate_revaluation_weekly",
],
"daily_long": [
"erpnext.accounts.doctype.process_subscription.process_subscription.create_subscription_process",
"erpnext.setup.doctype.email_digest.email_digest.send",
"erpnext.manufacturing.doctype.bom_update_tool.bom_update_tool.auto_update_latest_price_in_all_boms",
"erpnext.crm.utils.open_leads_opportunities_based_on_todays_event",