Merge pull request #660 from GSA/remove-template-message-limit

Remove unused variable

app/service/rest.py

@@ -260,10 +260,7 @@ def update_service(service_id):
         send_notification_to_service_users(
             service_id=service_id,
             template_id=current_app.config["SERVICE_NOW_LIVE_TEMPLATE_ID"],
-            personalisation={
-                "service_name": current_data["name"],
-                "message_limit": "{:,}".format(current_data["message_limit"]),
-            },
+            personalisation={"service_name": current_data["name"]},
             include_user_fields=["name"],
         )
 

migrations/versions/0407_fix_preferred_timezone.py

@@ -10,8 +10,9 @@ revision = "0407_fix_preferred_timezone"
 
 
 def upgrade():
-    op.execute("update users set preferred_timezone='US/Eastern' where preferred_timezone=''")
-
+    op.execute(
+        "update users set preferred_timezone='US/Eastern' where preferred_timezone=''"
+    )
 
 
 def downgrade():

migrations/versions/0408_fix_timezone_again.py

@@ -10,8 +10,9 @@ revision = "0408_fix_timezone_again"
 
 
 def upgrade():
-    op.execute("update users set preferred_timezone='US/Eastern' where (preferred_timezone='') is not false")
-
+    op.execute(
+        "update users set preferred_timezone='US/Eastern' where (preferred_timezone='') is not false"
+    )
 
 
 def downgrade():

tests/app/service/test_rest.py

@@ -2399,10 +2399,7 @@ def test_update_service_calls_send_notification_as_service_becomes_live(
     send_notification_mock.assert_called_once_with(
         service_id=restricted_service.id,
         template_id="618185c6-3636-49cd-b7d2-6f6f5eb3bdde",
-        personalisation={
-            "service_name": restricted_service.name,
-            "message_limit": "1,000",
-        },
+        personalisation={"service_name": restricted_service.name},
         include_user_fields=["name"],
     )