Task/remove seed (#372)

* Remove seeded_at key

* fix

Author: jzbahrai

.github/actions/waffles/requirements.txt

@@ -2,4 +2,4 @@ docopt==0.6.2
 Flask==2.3.3
 markupsafe==2.1.5
 setuptools==75.6.0 # required for distutils in Python 3.12
-git+https://github.com/cds-snc/notifier-utils.git@53.2.3#egg=notifications-utils
+git+https://github.com/cds-snc/notifier-utils.git@53.2.4#egg=notifications-utils

notifications_utils/clients/redis/annual_limit.py

@@ -47,6 +47,7 @@
 EMAIL_DELIVERED_TODAY = "email_delivered_today"
 SMS_FAILED_TODAY = "sms_failed_today"
 EMAIL_FAILED_TODAY = "email_failed_today"
+SEEDED_AT = "seeded_at"
 TOTAL_SMS_FISCAL_YEAR_TO_YESTERDAY = "total_sms_fiscal_year_to_yesterday"
 TOTAL_EMAIL_FISCAL_YEAR_TO_YESTERDAY = "total_email_fiscal_year_to_yesterday"
 
@@ -64,7 +65,6 @@
 NEAR_EMAIL_LIMIT = "near_email_limit"
 OVER_SMS_LIMIT = "over_sms_limit"
 OVER_EMAIL_LIMIT = "over_email_limit"
-SEEDED_AT = "seeded_at"
 
 STATUS_FIELDS = [NEAR_SMS_LIMIT, NEAR_EMAIL_LIMIT, OVER_SMS_LIMIT, OVER_EMAIL_LIMIT]
 
@@ -194,7 +194,8 @@ def reset_all_notification_counts(self, service_ids=None):
             if not service_ids
             else [annual_limit_notifications_v2_key(service_id) for service_id in service_ids]
         )
-        self._redis_client.delete_hash_fields(hashes=hashes, fields=NOTIFICATION_FIELDS_V2)
+        # We also want to remove the seeded_at field from the notifications_v2 hash
+        self._redis_client.delete_hash_fields(hashes=hashes, fields=NOTIFICATION_FIELDS_V2 + [SEEDED_AT])
         # TODO: Remove the else once all services have been migrated to the new Redis structure
         hashes = (
             annual_limit_notifications_key("*")

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "notifications-utils"
-version = "53.2.3"
+version = "53.2.4"
 description = "Shared python code for Notification - Provides logging utils etc."
 authors = ["Canadian Digital Service"]
 license = "MIT license"