diff --git a/src/membership/emails.py b/src/membership/emails.py index 46594ef..a257bd1 100644 --- a/src/membership/emails.py +++ b/src/membership/emails.py @@ -36,7 +36,7 @@ class BaseEmail(EmailMessage): template = "membership/email/base.txt" # Optional: Set to a template path for subject template_subject = None - default_subject : StrOrPromise = "SET SUBJECT HERE" + default_subject: StrOrPromise = "SET SUBJECT HERE" def __init__(self, request: HttpRequest, *args, **kwargs) -> None: # noqa: ANN002, ANN003 self.context = kwargs.pop("context", {}) diff --git a/src/membership/permissions.py b/src/membership/permissions.py index d665233..c90ffd9 100644 --- a/src/membership/permissions.py +++ b/src/membership/permissions.py @@ -2,8 +2,8 @@ from __future__ import annotations -from typing import TYPE_CHECKING from dataclasses import dataclass +from typing import TYPE_CHECKING from django.contrib.auth.models import Permission as DjangoPermission from django.contrib.contenttypes.models import ContentType @@ -12,7 +12,7 @@ from django.utils.translation import gettext_lazy as _ if TYPE_CHECKING: from django_stubs_ext import StrOrPromise -PERMISSIONS : list[Permission] = [] +PERMISSIONS: list[Permission] = [] def persist_permissions(*args, **kwargs) -> None: # type: ignore[no-untyped-def] # noqa: ANN002, ANN003 diff --git a/src/membership/views.py b/src/membership/views.py index cae38a8..504de80 100644 --- a/src/membership/views.py +++ b/src/membership/views.py @@ -25,9 +25,9 @@ from .selectors import get_memberships from .selectors import get_subscription_periods if TYPE_CHECKING: - from utils.types import AuthenticatedHttpRequest from django.http import HttpRequest from django.http import HttpResponse + from utils.types import AuthenticatedHttpRequest member_view = namespaced_decorator_factory(namespace="member", base_path="membership") diff --git a/src/project/settings.py b/src/project/settings.py index 32747f4..8289af4 100644 --- a/src/project/settings.py +++ b/src/project/settings.py @@ -108,7 +108,7 @@ AUTHENTICATION_BACKENDS = ( WSGI_APPLICATION = "project.wsgi.application" -AUTH_PASSWORD_VALIDATORS : list[dict[str, str]] = [] +AUTH_PASSWORD_VALIDATORS: list[dict[str, str]] = [] LANGUAGE_CODE = "da-dk" diff --git a/src/services/registry.py b/src/services/registry.py index 85e3afd..b8e51fb 100644 --- a/src/services/registry.py +++ b/src/services/registry.py @@ -41,7 +41,7 @@ class ServiceInterface(Interface): request_types: list[ServiceRequests] = DEFAULT_SERVICE_REQUEST_TYPES - subscribe_fields: tuple[tuple[str, forms.Field],...] = () + subscribe_fields: tuple[tuple[str, forms.Field], ...] = () def get_form_class(self) -> type: """Get the form class for the service.""" diff --git a/src/services/views.py b/src/services/views.py index d529804..db20756 100644 --- a/src/services/views.py +++ b/src/services/views.py @@ -14,8 +14,8 @@ from services.registry import ServiceRegistry if TYPE_CHECKING: from django.contrib.auth.models import User - from utils.types import AuthenticatedHttpRequest from django.http import HttpResponse + from utils.types import AuthenticatedHttpRequest services_view = namespaced_decorator_factory( namespace="services", @@ -34,7 +34,9 @@ def services_overview(request: AuthenticatedHttpRequest) -> HttpResponse: active_service_classes = [service.__class__ for service in active_services] - non_active_services = [service for _, service in ServiceRegistry.get_items() if service not in active_service_classes] + non_active_services = [ + service for _, service in ServiceRegistry.get_items() if service not in active_service_classes + ] context = { "non_active_services": non_active_services, diff --git a/src/utils/management/commands/bootstrap_dev_data.py b/src/utils/management/commands/bootstrap_dev_data.py index e95476f..5490a85 100644 --- a/src/utils/management/commands/bootstrap_dev_data.py +++ b/src/utils/management/commands/bootstrap_dev_data.py @@ -1,10 +1,11 @@ """Command to bootstrap development data.""" from datetime import timedelta -from django.utils import timezone -from django.db.backends.postgresql.psycopg_any import DateRange + from django.contrib.auth.models import User from django.core.management import BaseCommand +from django.db.backends.postgresql.psycopg_any import DateRange +from django.utils import timezone from membership.models import SubscriptionPeriod diff --git a/src/utils/types.py b/src/utils/types.py index 01553c5..efd8093 100644 --- a/src/utils/types.py +++ b/src/utils/types.py @@ -7,5 +7,4 @@ from django.http import HttpRequest class AuthenticatedHttpRequest(HttpRequest): """HttpRequest with an authenticated user.""" - # XXX(reynir): Should this be Member instead?! user: User diff --git a/src/utils/view_utils.py b/src/utils/view_utils.py index 7a47f87..098fc2e 100644 --- a/src/utils/view_utils.py +++ b/src/utils/view_utils.py @@ -89,7 +89,6 @@ class RenderConfig: for obj in objects: with queries_disabled(): row = Row( - # XXX(reynir): we never use the key data={column[0]: getattr(obj, column[0]) for column in columns}, actions=[action.render(obj) for action in row_actions], )