From cec87a9de04b7e5953843201954b4cafeb0b7327 Mon Sep 17 00:00:00 2001 From: Katie Smith Date: Fri, 12 Jul 2019 11:23:45 +0100 Subject: [PATCH] Delete unused code * The `_should_record_notification_in_history_table` function stopped being used in this commit: c23ae15f32c4673b9d5a40f338e9e1eb87190430 * `NOTIFICATIONS_ALERT` stopped being used in this commit: 5aa37f09b63e7d85eac9224ed0084a2501ed3e84 --- app/config.py | 1 - app/dao/notifications_dao.py | 8 -------- 2 files changed, 9 deletions(-) diff --git a/app/config.py b/app/config.py index 0b074e8c6..25117ca07 100644 --- a/app/config.py +++ b/app/config.py @@ -293,7 +293,6 @@ class Config(object): } CELERY_QUEUES = [] - NOTIFICATIONS_ALERT = 5 # five mins FROM_NUMBER = 'development' STATSD_HOST = os.getenv('STATSD_HOST') diff --git a/app/dao/notifications_dao.py b/app/dao/notifications_dao.py index 882189ae3..84f9e1249 100644 --- a/app/dao/notifications_dao.py +++ b/app/dao/notifications_dao.py @@ -79,14 +79,6 @@ def dao_create_notification(notification): db.session.add(notification) -def _should_record_notification_in_history_table(notification): - if notification.api_key_id and notification.key_type == KEY_TYPE_TEST: - return False - if notification.service.research_mode: - return False - return True - - def _decide_permanent_temporary_failure(current_status, status): # Firetext will send pending, then send either succes or fail. # If we go from pending to delivered we need to set failure type as temporary-failure