From c1ff21b8770050595ee4681d6aa45b78d47dc491 Mon Sep 17 00:00:00 2001 From: vrashank Date: Sat, 26 Apr 2025 10:14:08 +0000 Subject: [PATCH] Fixes the bug of required positionalo arg shipped date and ack type --- sps_integration/sps_integration/order_acknowledgement.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sps_integration/sps_integration/order_acknowledgement.py b/sps_integration/sps_integration/order_acknowledgement.py index 52bfe6d..35e6439 100644 --- a/sps_integration/sps_integration/order_acknowledgement.py +++ b/sps_integration/sps_integration/order_acknowledgement.py @@ -94,7 +94,7 @@ def format_sales_order_ack(data: dict, schedule_shipped_date: str, ack_typ: str) -def sales_order_acknowledgement(data: dict , path: str): +def sales_order_acknowledgement(data: dict , path: str, schedule_shipped_date: str, ack_type:str): """Send Sales Order Acknowledgment to SPS Commerce API.""" try: doc = frappe.get_doc("SPS Integration Settings", {"enabled":1}) @@ -105,7 +105,7 @@ def sales_order_acknowledgement(data: dict , path: str): "Authorization": f"Bearer {access_token}" } - formatted_data = format_sales_order_ack(data) + formatted_data = format_sales_order_ack(data, schedule_shipped_date, ack_type) if not formatted_data: return @@ -157,7 +157,7 @@ def submit_order_acknowledgement(path: str, schedule_shipped_date: str, ack_type formatted_data = format_sales_order_ack(response.json(),schedule_shipped_date, ack_type) frappe.log_error(title="Formatted SO Ack Payload", message=f"{formatted_data}") # Sales order ack functionality - ack_response = sales_order_acknowledgement(data=formatted_data,path=path) + ack_response = sales_order_acknowledgement(data=formatted_data,path=path, schedule_shipped_date=schedule_shipped_date, ack_type=ack_type) # ack_response = { # "path":"/in/ACK_a86e370c-43b9-3db1-b6bd-6289a132448f}-d76e3c43e0024359960669e9855509ce.dat", # "url":"https://api.spscommerce.com/transactions/v5/data/in/ACK_a86e370c-43b9-3db1-b6bd-6289a132448f}-d76e3c43e0024359960669e9855509ce.dat"}