refactor the update payment function

commcare_connect/opportunity/visit_import.py

@@ -271,12 +271,18 @@ def _bulk_update_payments(opportunity: Opportunity, imported_data: Dataset) -> P
             payment = Payment.objects.create(opportunity_access=access, amount=amount, amount_usd=amount_usd)
             seen_users.add(username)
             payment_ids.append(payment.pk)
-            update_work_payment_date(access)
+        process_work_payment_dates(users)
     missing_users = set(usernames) - seen_users
     send_payment_notification.delay(opportunity.id, payment_ids)
     return PaymentImportStatus(seen_users, missing_users)
 
 
+def process_work_payment_dates(accesses: list):
+    with transaction.atomic():
+        for oa in accesses:
+            update_work_payment_date(oa)
+
+
 def _cache_key(currency_code, date=None):
     return [currency_code, date.toordinal() if date else None]