formatting changes

.ds.baseline

@@ -161,7 +161,7 @@
         "filename": "app/config.py",
         "hashed_secret": "577a4c667e4af8682ca431857214b3a920883efc",
         "is_verified": false,
-        "line_number": 123,
+        "line_number": 121,
         "is_secret": false
       }
     ],
@@ -634,5 +634,5 @@
       }
     ]
   },
-  "generated_at": "2025-07-22T17:07:31Z"
+  "generated_at": "2025-07-29T20:46:51Z"
 }

app/config.py

@@ -16,9 +16,7 @@ class Config(object):
     API_PUBLIC_WS_URL = getenv("API_PUBLIC_WS_URL", "localhost")
 
     ADMIN_BASE_URL = getenv("ADMIN_BASE_URL", "http://localhost:6012")
-    HEADER_COLOUR = (
+    HEADER_COLOUR = "#81878b"  # mix of dark-grey and mid-grey
-        "#81878b"  # mix of dark-grey and mid-grey
-    )
     LOGO_CDN_DOMAIN = (
         "static-logos.notifications.service.gov.uk"  # TODO use our own CDN
     )

app/main/views/manage_users.py

@@ -73,7 +73,9 @@ def invite_user(service_id, user_id=None):
     else:
         user_to_invite = None
 
-    service_has_email_auth = current_service.has_permission(ServicePermission.EMAIL_AUTH)
+    service_has_email_auth = current_service.has_permission(
+        ServicePermission.EMAIL_AUTH
+    )
     if not service_has_email_auth:
         form.login_authentication.data = "sms_auth"
 
@@ -116,7 +118,9 @@ def invite_user(service_id, user_id=None):
 @main.route("/services/<uuid:service_id>/users/<uuid:user_id>", methods=["GET", "POST"])
 @user_has_permissions(ServicePermission.MANAGE_SERVICE)
 def edit_user_permissions(service_id, user_id):
-    service_has_email_auth = current_service.has_permission(ServicePermission.EMAIL_AUTH)
+    service_has_email_auth = current_service.has_permission(
+        ServicePermission.EMAIL_AUTH
+    )
     user = current_service.get_team_member(user_id)
 
     mobile_number = None

app/main/views/notifications.py

@@ -94,7 +94,9 @@ def view_notification(service_id, notification_id, error_message=None):
         updated_at=notification["sent_at"],
         help=get_help_argument(),
         notification_id=notification["id"],
-        can_receive_inbound=(current_service.has_permission(ServicePermission.INBOUND_SMS)),
+        can_receive_inbound=(
+            current_service.has_permission(ServicePermission.INBOUND_SMS)
+        ),
         sent_with_test_key=(notification.get("key_type") == KEY_TYPE_TEST),
         back_link=back_link,
     )

app/main/views/send.py

@@ -560,7 +560,9 @@ def _check_messages(service_id, template_id, upload_id, preview_row, **kwargs):
         max_errors_shown=50,
         guestlist=allow_list,
         remaining_messages=remaining_messages,
-        allow_international_sms=current_service.has_permission(ServicePermission.INTERNATIONAL_SMS),
+        allow_international_sms=current_service.has_permission(
+            ServicePermission.INTERNATIONAL_SMS
+        ),
     )
 
     if request.args.get("from_test"):

app/utils/user_permissions.py

@@ -4,7 +4,10 @@ from app.enums import ServicePermission
 
 permission_mappings = {
     # TODO: consider turning off email-sending permissions during SMS pilot
-    ServicePermission.SEND_MESSAGES: [ServicePermission.SEND_TEXTS, ServicePermission.SEND_EMAILS],
+    ServicePermission.SEND_MESSAGES: [
+        ServicePermission.SEND_TEXTS,
+        ServicePermission.SEND_EMAILS,
+    ],
     ServicePermission.MANAGE_TEMPLATES: [ServicePermission.MANAGE_TEMPLATES],
     ServicePermission.MANAGE_SERVICE: [
         ServicePermission.MANAGE_USERS,

tests/app/main/views/service_settings/test_service_setting_permissions.py

@@ -202,7 +202,11 @@ def test_service_setting_link_toggles_index_error(
     ("permissions", "permissions_text", "visible"),
     [
         ("sms", "inbound SMS", True),
-        (ServicePermission.INBOUND_SMS, "inbound SMS", False),  # no sms parent permission
+        (
+            ServicePermission.INBOUND_SMS,
+            "inbound SMS",
+            False,
+        ),  # no sms parent permission
         # also test no permissions set
         ("", "inbound SMS", False),
     ],

tests/app/main/views/test_dashboard.py

@@ -966,7 +966,7 @@ def test_menu_manage_service(
             ServicePermission.VIEW_ACTIVITY,
             ServicePermission.MANAGE_TEMPLATES,
             ServicePermission.MANAGE_USERS,
-            ServicePermission.MANAGE_SETTINGS
+            ServicePermission.MANAGE_SETTINGS,
         ],
     )
     page = str(page)

tests/app/utils/test_user.py

@@ -36,7 +36,11 @@ def test_permissions(
     request.view_args.update({"service_id": "foo"})
 
     api_user_active["permissions"] = {
-        "foo": [ServicePermission.MANAGE_USERS, ServicePermission.MANAGE_TEMPLATES, ServicePermission.MANAGE_SETTINGS]
+        "foo": [
+            ServicePermission.MANAGE_USERS,
+            ServicePermission.MANAGE_TEMPLATES,
+            ServicePermission.MANAGE_SETTINGS,
+        ]
     }
     api_user_active["services"] = ["foo", "bar"]
 
@@ -66,7 +70,11 @@ def test_permissions_forbidden(
     request.view_args.update({"service_id": "foo"})
 
     api_user_active["permissions"] = {
-        "foo": [ServicePermission.MANAGE_USERS, ServicePermission.MANAGE_TEMPLATES, ServicePermission.MANAGE_SETTINGS]
+        "foo": [
+            ServicePermission.MANAGE_USERS,
+            ServicePermission.MANAGE_TEMPLATES,
+            ServicePermission.MANAGE_SETTINGS,
+        ]
     }
     api_user_active["services"] = ["foo", "bar"]
 
@@ -179,7 +187,11 @@ def test_user_with_no_permissions_to_service_goes_to_templates(
     api_user_active,
 ):
     api_user_active["permissions"] = {
-        "foo": [ServicePermission.MANAGE_USERS, ServicePermission.MANAGE_TEMPLATES, ServicePermission.MANAGE_SETTINGS]
+        "foo": [
+            ServicePermission.MANAGE_USERS,
+            ServicePermission.MANAGE_TEMPLATES,
+            ServicePermission.MANAGE_SETTINGS,
+        ]
     }
     api_user_active["services"] = ["foo", "bar"]
     client_request.login(api_user_active)