Check if order is already settled.
This commit is contained in:
parent
1dddbf5ef1
commit
964682716f
2 changed files with 24 additions and 6 deletions
|
|
@ -52,9 +52,7 @@ def generate_um_report(start_date: datetime.datetime, end_date: datetime.datetim
|
||||||
"status": "processing,completed",
|
"status": "processing,completed",
|
||||||
},
|
},
|
||||||
).json()
|
).json()
|
||||||
|
|
||||||
orders_in_date_range = [Order.from_api_response(order_raw_data) for order_raw_data in orders_in_date_range]
|
orders_in_date_range = [Order.from_api_response(order_raw_data) for order_raw_data in orders_in_date_range]
|
||||||
|
|
||||||
logger.info(f"Received {len(orders_in_date_range)} orders.")
|
logger.info(f"Received {len(orders_in_date_range)} orders.")
|
||||||
|
|
||||||
skus_to_keep = ["TEE-05-BBO-BLACK"]
|
skus_to_keep = ["TEE-05-BBO-BLACK"]
|
||||||
|
|
@ -67,15 +65,17 @@ def generate_um_report(start_date: datetime.datetime, end_date: datetime.datetim
|
||||||
|
|
||||||
logger.info("Checking if all orders have the sats_received entry filled in.")
|
logger.info("Checking if all orders have the sats_received entry filled in.")
|
||||||
orders_without_sats_received = find_orders_without_sats_received(relevant_orders)
|
orders_without_sats_received = find_orders_without_sats_received(relevant_orders)
|
||||||
|
|
||||||
if orders_without_sats_received:
|
if orders_without_sats_received:
|
||||||
logger.warning(
|
logger.warning(
|
||||||
f"There are {len(orders_without_sats_received)} orders without a properly filled sats_received entry.")
|
f"There are {len(orders_without_sats_received)} orders without a properly filled sats_received entry.")
|
||||||
logger.warning(f"See details below.")
|
logger.warning(f"See details below.")
|
||||||
logger.warning(orders_without_sats_received)
|
logger.warning(orders_without_sats_received)
|
||||||
raise ValueError("Not all orders have sats_received. Can't compute sats owed without that.")
|
raise ValueError("Not all orders have sats_received. Can't compute sats owed without that.")
|
||||||
|
|
||||||
logger.info("Success, all orders have sats_received filled in.")
|
logger.info("Success, all orders have sats_received filled in.")
|
||||||
|
|
||||||
|
logger.info("Removing settled orders.")
|
||||||
|
unsettled_orders = filter_settled_orders(relevant_orders)
|
||||||
|
logger.info(f"Kept {len(unsettled_orders)} unsettled orders.")
|
||||||
# Fetch orders:
|
# Fetch orders:
|
||||||
# - Between specific dates
|
# - Between specific dates
|
||||||
# - That contain the hardcoded products
|
# - That contain the hardcoded products
|
||||||
|
|
@ -114,3 +114,13 @@ def find_orders_without_sats_received(orders: List[Order]) -> List[Order]:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
return orders_without_sats_received
|
return orders_without_sats_received
|
||||||
|
|
||||||
|
|
||||||
|
def filter_settled_orders(orders: List[Order]) -> List[Order]:
|
||||||
|
unsettled_orders = []
|
||||||
|
|
||||||
|
for order in orders:
|
||||||
|
if not order.is_settled_with_um():
|
||||||
|
unsettled_orders.append(order)
|
||||||
|
|
||||||
|
return unsettled_orders
|
||||||
|
|
|
||||||
|
|
@ -9,6 +9,10 @@ class Order:
|
||||||
def __getitem__(self, item):
|
def __getitem__(self, item):
|
||||||
return self.raw_data[item]
|
return self.raw_data[item]
|
||||||
|
|
||||||
|
@property
|
||||||
|
def meta_data_entries(self):
|
||||||
|
return {meta_data_entry["key"]: meta_data_entry["value"] for meta_data_entry in self.raw_data["meta_data"]}
|
||||||
|
|
||||||
def contains_sku(self, sku: str) -> bool:
|
def contains_sku(self, sku: str) -> bool:
|
||||||
for item in self["line_items"]:
|
for item in self["line_items"]:
|
||||||
if item["sku"] == sku:
|
if item["sku"] == sku:
|
||||||
|
|
@ -16,11 +20,15 @@ class Order:
|
||||||
return False
|
return False
|
||||||
|
|
||||||
def contains_meta_data_entry(self, meta_data_entry_key: str) -> bool:
|
def contains_meta_data_entry(self, meta_data_entry_key: str) -> bool:
|
||||||
meta_data_entries = {meta_data_entry["key"]: meta_data_entry["value"] for meta_data_entry in self["meta_data"]}
|
if meta_data_entry_key in self.meta_data_entries.keys():
|
||||||
if meta_data_entry_key in meta_data_entries.keys():
|
|
||||||
return True
|
return True
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
def is_settled_with_um(self):
|
||||||
|
is_settled = self.meta_data_entries.get("is_settled_with_um", None)
|
||||||
|
|
||||||
|
return bool(is_settled)
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def from_api_response(cls, raw_data) -> "Order":
|
def from_api_response(cls, raw_data) -> "Order":
|
||||||
return Order(raw_data)
|
return Order(raw_data)
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue