Process transactions other than Subscription payments

Author: rennerocha

thebook/webhooks/paypal/services.py

@@ -97,11 +97,12 @@ def fetch_transactions(start_date: datetime.date, end_date: datetime.date):
                 "transaction_info.transaction_subject", transaction
             )
 
-        paypal_reference_id = jmespath.search(
-            "transaction_info.paypal_reference_id", transaction
-        )
-        if paypal_reference_id:
-            # This flow only applies to membership subscriptions
+        if transaction_type == "T0002":
+            # This flow only applies to Subscription payment
+            # https://developer.paypal.com/docs/transaction-search/transaction-event-codes/
+            paypal_reference_id = jmespath.search(
+                "transaction_info.paypal_reference_id", transaction
+            )
             response = requests.get(
                 f"{settings.PAYPAL_API_BASE_URL}/v1/billing/subscriptions/{paypal_reference_id}",
                 headers={"Authorization": f"Bearer {access_token}"},

thebook/webhooks/tests/paypal/sample_payloads/reporting_transactions__one_common_transaction.json

@@ -0,0 +1,51 @@
+{
+  "transaction_details": [
+    {
+      "transaction_info": {
+        "paypal_account_id": "XPZJRN6WX8Q7Y",
+        "transaction_id": "3DJ715755L433650N",
+        "paypal_reference_id": "11J12750B64965920",
+        "paypal_reference_id_type": "TXN",
+        "transaction_event_code": "T0006",
+        "transaction_initiation_date": "2026-03-09T18:41:22Z",
+        "transaction_updated_date": "2026-03-09T18:41:22Z",
+        "transaction_amount": {
+          "currency_code": "BRL",
+          "value": "51.00"
+        },
+        "fee_amount": {
+          "currency_code": "BRL",
+          "value": "-3.04"
+        },
+        "transaction_status": "S",
+        "transaction_subject": "Order 97XKB for Tosconf[6]",
+        "ending_balance": {
+          "currency_code": "BRL",
+          "value": "47.96"
+        },
+        "available_balance": {
+          "currency_code": "BRL",
+          "value": "47.96"
+        },
+        "custom_field": "Order 2026-97XKB",
+        "protection_eligibility": "01",
+        "instrument_type": "PayPal",
+        "instrument_sub_type": "PayPal Wallet"
+      }
+    }
+  ],
+  "account_number": "2LBBYEY3XL7P8",
+  "start_date": "2026-03-09T00:00:00Z",
+  "end_date": "2026-03-09T20:29:59Z",
+  "last_refreshed_datetime": "2026-03-09T20:29:59Z",
+  "page": 1,
+  "total_items": 1,
+  "total_pages": 1,
+  "links": [
+    {
+      "href": "https://api.paypal.com/v1/reporting/transactions?start_date=2026-03-09T00%3A00%3A00-00%3A00&end_date=2026-03-10T00%3A00%3A00-00%3A00&page_size=100&page=1",
+      "rel": "self",
+      "method": "GET"
+    }
+  ]
+}

thebook/webhooks/tests/paypal/test_services__fetch_transactions.py

@@ -46,6 +46,17 @@ def paypal__oauth2_token():
     return {"access_token": "test-access-token"}
 
 
+@pytest.fixture
+def reporting_transactions__one_common_transaction():
+    with open(
+        Path(
+            SAMPLE_PAYLOADS_DIR / "reporting_transactions__one_common_transaction.json"
+        ),
+        "r",
+    ) as payload:
+        return payload.read()
+
+
 @pytest.fixture
 def paypal__brl_payload__subscription():
     with open(
@@ -148,6 +159,52 @@ def test_fetch_one_transaction(
     assert transactions[1].created_by == user
 
 
+@responses.activate
+def test_fetch_one_common_transaction(
+    db,
+    paypal_bank_account,
+    user,
+    bank_fee_category,
+    paypal__oauth2_token,
+    reporting_transactions__one_common_transaction,
+):
+    responses.add(
+        responses.POST,
+        f"{settings.PAYPAL_API_BASE_URL}/v1/oauth2/token",
+        json=paypal__oauth2_token,
+    )
+    responses.add(
+        responses.GET,
+        f"{settings.PAYPAL_API_BASE_URL}/v1/reporting/transactions",
+        body=reporting_transactions__one_common_transaction,
+        content_type="application/json",
+    )
+
+    transactions = fetch_transactions(
+        start_date=datetime.date(2026, 3, 9), end_date=datetime.date(2026, 3, 10)
+    )
+
+    assert len(transactions) == 2
+
+    assert transactions[0].id is None
+    assert transactions[0].reference == "3DJ715755L433650N"
+    assert transactions[0].date == datetime.date(2026, 3, 9)
+    assert transactions[0].description == "Order 97XKB for Tosconf[6]"
+    assert transactions[0].amount == Decimal("51")
+    assert transactions[0].bank_account == paypal_bank_account
+    assert transactions[0].category is None
+    assert transactions[0].created_by == user
+
+    assert transactions[1].id is None
+    assert transactions[1].reference == "3DJ715755L433650N-T"
+    assert transactions[1].date == datetime.date(2026, 3, 9)
+    assert transactions[1].description == "Taxa Paypal - Order 97XKB for Tosconf[6]"
+    assert transactions[1].amount == Decimal("-3.04")
+    assert transactions[1].bank_account == paypal_bank_account
+    assert transactions[1].category == bank_fee_category
+    assert transactions[1].created_by == user
+
+
 @responses.activate
 def test_fetch_multiple_transactions(
     db,