Remove RECONCILE_IN_WEBHOOK feature flag

It worked fine last time and I'd prefer not to have unused options
emfcamp · Jan 23, 2024 · e876978 · e876978
1 parent 39593da
commit e876978
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 7 deletions.
diff --git a/apps/payments/wise.py b/apps/payments/wise.py
@@ -8,7 +8,6 @@
 from models.payment import BankAccount, BankTransaction
 from . import payments
 from .banktransfer import reconcile_txns
-from ..common import feature_enabled
 from main import db, wise
 
 logger = logging.getLogger(__name__)
@@ -177,9 +176,8 @@ def sync_wise_statement(profile_id, borderless_account_id, currency):
     logger.info("Imported %s transactions", len(txns))
     db.session.commit()
 
-    if feature_enabled("RECONCILE_IN_WEBHOOK"):
-        logger.info("Attempting reconciliation")
-        reconcile_txns(txns, doit=True)
+    logger.info("Reconciling...")
+    reconcile_txns(txns, doit=True)
 
 
 def wise_business_profile():

diff --git a/config/development-example.cfg b/config/development-example.cfg
@@ -50,7 +50,6 @@ SCHEDULE = False
 RADIO = False
 ISSUE_TICKETS = False
 REFUND_REQUESTS = False
-RECONCILE_IN_WEBHOOK = True
 VOLUNTEER_SITE = True
 VOLUNTEERS_SIGNUP = True
 VOLUNTEERS_SCHEDULE = True

diff --git a/config/kelvin.cfg b/config/kelvin.cfg
@@ -58,7 +58,6 @@ CFP_CLOSED = False
 VOLUNTEERS_SCHEDULE = True
 RADIO = False
 ISSUE_TICKETS = False
-RECONCILE_IN_WEBHOOK = True
 LINE_UP = False
 SCHEDULE = False
 REFUND_REQUESTS = False

diff --git a/models/feature_flag.py b/models/feature_flag.py
@@ -10,7 +10,6 @@
     "CFP_CLOSED",
     "CFP_FINALISE",
     "ISSUE_TICKETS",
-    "RECONCILE_IN_WEBHOOK",
     "RADIO",
     "LINE_UP",
     "LIGHTNING_TALKS",