Skip to content

Commit

Permalink
fix: avoid duplicate logs for old order sync
Browse files Browse the repository at this point in the history
  • Loading branch information
ankush committed Oct 24, 2023
1 parent 094134c commit 9f90628
Showing 1 changed file with 9 additions and 18 deletions.
27 changes: 9 additions & 18 deletions ecommerce_integrations/shopify/order.py
Original file line number Diff line number Diff line change
Expand Up @@ -399,26 +399,17 @@ def sync_old_orders():
if not cint(shopify_setting.sync_old_orders):
return

try:
orders = _fetch_old_orders(shopify_setting.old_orders_from, shopify_setting.old_orders_to)

for order in orders:
log = create_shopify_log(
method=EVENT_MAPPER["orders/create"], request_data=json.dumps(order), make_new=True
)
sync_sales_order(order, request_id=log.name)
orders = _fetch_old_orders(shopify_setting.old_orders_from, shopify_setting.old_orders_to)

shopify_setting = frappe.get_doc(SETTING_DOCTYPE)
shopify_setting.sync_old_orders = 0
shopify_setting.save()

create_shopify_log(
status="Success", method="ecommerce_integrations.shopify.order.sync_old_orders"
)
except Exception as e:
create_shopify_log(
status="Error", method="ecommerce_integrations.shopify.order.sync_old_orders", exception=e
for order in orders:
log = create_shopify_log(
method=EVENT_MAPPER["orders/create"], request_data=json.dumps(order), make_new=True
)
sync_sales_order(order, request_id=log.name)

shopify_setting = frappe.get_doc(SETTING_DOCTYPE)
shopify_setting.sync_old_orders = 0
shopify_setting.save()


def _fetch_old_orders(from_time, to_time):
Expand Down

0 comments on commit 9f90628

Please sign in to comment.