fix(worker): only count successful payments toward total spend (#31)

Failed and refunded charges were being summed into the 'spent on Nitro' total, inflating the figure (sometimes 4x). Filter on Discord's Payment.status enum to keep only status=1 (SUCCESSFUL).

Closes dumpus-app/dumpus-app#261.

Author: Mr-KayJayDee

src/tasks.py

@@ -237,7 +237,13 @@ def read_analytics_file(package_status_id, package_id, link, session):
             with zip.open(payments_path) as f:
                 payment_records = orjson.loads(f.read()).get('records', [])
 
+        # Only count successful payments. Discord's Payment.status enum:
+        # 1 = SUCCESSFUL, 2 = FAILED, 3 = REFUNDED, 4 = REFUND_PENDING,
+        # 5 = PARTIALLY_REFUNDED, 6 = PENDING. Without this filter, failed
+        # and refunded charges inflate the "spent on Nitro" total.
         for payment in payment_records:
+            if payment.get('status') != 1:
+                continue
             payments.append({
                 'id': payment['id'],
                 'amount': payment['amount'],