Wagtail 7.2 maintenance

.github/dependabot.yml

@@ -9,20 +9,19 @@ updates:
   ignore:
   - dependency-name: flake8
     versions:
-    - 3.8.4
-    - 3.9.0
+    - 7.3.0
   - dependency-name: coverage
     versions:
-    - "5.4"
+    - "7.10.1"
   - dependency-name: pytest
     versions:
-    - 6.2.2
+    - 8.4.1
   - dependency-name: isort
     versions:
-    - 5.7.0
+    - 6.0.1
   - dependency-name: django-debug-toolbar
     versions:
-    - "3.2"
+    - "6.0"
   - dependency-name: pytest-django
     versions:
-    - 4.1.0
+    - 4.11.1

.github/workflows/python-release.yml

@@ -9,18 +9,14 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-
       - name: Set up Python 3.11
         uses: actions/setup-python@v4
         with:
           python-version: 3.11
-
       - name: Install build requirements
         run: python -m pip install wheel
-
       - name: Build package
         run: python setup.py sdist bdist_wheel
-
       - name: Publish package
         uses: pypa/gh-action-pypi-publish@release/v1
         with:

.github/workflows/python-tox.yml

@@ -7,20 +7,17 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python: ["3.8", "3.9", "3.10", "3.11"]
+        python: ['3.10', '3.11', '3.12', '3.13', '3.14']
     steps:
-      - uses: actions/checkout@v4
-
-      - name: Set up Python ${{ matrix.python }}
-        uses: actions/setup-python@v4
-        with:
-          python-version: ${{ matrix.python }}
-
-      - name: Install dependencies
-        run: |
-          python -m pip install --upgrade pip
-          python -m pip install tox tox-gh-actions
-
-      - name: Test with tox
-        run: |
+    - uses: actions/checkout@v4
+    - name: Set up Python ${{ matrix.python }}
+      uses: actions/setup-python@v5
+      with:
+        python-version: ${{ matrix.python }}
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        python -m pip install tox tox-gh-actions
+    - name: Test with tox
+      run: |
           tox

CHANGES

@@ -1,3 +1,11 @@
+Unreleased
+==========
+ - Add testing for Python 3.14 and Wagtail 7.2 
+ - Update dependencies to latest versions
+ - Update Wagtail supported versions to 6.3, 7.0 and 7.1 (Currently supported versions)
+ - Update Django supported versions to 4.2, 5.1 and 5.2 (Currently supported versions)
+ - Removed old code for handling no longer supported versions
+
 1.7.1 (2025-07-24)
 =================
  - Update python release workflow to use Python 3.11
@@ -6,7 +14,6 @@
 1.7.0 (2025-07-24)
 =================
  - Added support for wagtail 6.2 and above
-
 
 1.6.9 (2023-12-20)
 =================

sandbox/requirements.txt

@@ -1,4 +1,4 @@
-Django>=3.2
-wagtail>=4.1
-django-debug-toolbar==3.2.2
+Django>=4.2
+wagtail>=6.3
+django-debug-toolbar==4.4.6
 -e .[docs,test]

sandbox/sandbox/apps/user/models.py

@@ -1,42 +1,115 @@
+from django.apps import apps
+from django.contrib import auth
+from django.contrib.auth.hashers import make_password
 from django.contrib.auth.models import (
-    AbstractBaseUser, PermissionsMixin, UserManager)
+    AbstractBaseUser,
+    BaseUserManager,
+    PermissionsMixin,
+)
 from django.core.mail import send_mail
 from django.db import models
 from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 
 
+class UserManager(BaseUserManager):
+    use_in_migrations = True
+
+    def _create_user(self, email, password, **extra_fields):
+        """
+        Create and save a user with the given email and password.
+        """
+        email = self.normalize_email(email)
+        # Lookup the real model class from the global app registry so this
+        # manager method can be used in migrations. This is fine because
+        # managers are by definition working on the real model.
+        GlobalUserModel = apps.get_model(
+            self.model._meta.app_label, self.model._meta.object_name
+        )
+        user = self.model(email=email, **extra_fields)
+        user.password = make_password(password)
+        user.save(using=self._db)
+        return user
+
+    def create_user(self, email=None, password=None, **extra_fields):
+        extra_fields.setdefault("is_staff", False)
+        extra_fields.setdefault("is_superuser", False)
+        return self._create_user(email, password, **extra_fields)
+
+    def create_superuser(self, email=None, password=None, **extra_fields):
+        extra_fields.setdefault("is_staff", True)
+        extra_fields.setdefault("is_superuser", True)
+
+        if extra_fields.get("is_staff") is not True:
+            raise ValueError("Superuser must have is_staff=True.")
+        if extra_fields.get("is_superuser") is not True:
+            raise ValueError("Superuser must have is_superuser=True.")
+
+        return self._create_user(email, password, **extra_fields)
+
+    def with_perm(
+        self, perm, is_active=True, include_superusers=True, backend=None, obj=None
+    ):
+        if backend is None:
+            backends = auth._get_backends(return_tuples=True)
+            if len(backends) == 1:
+                backend, _ = backends[0]
+            else:
+                raise ValueError(
+                    "You have multiple authentication backends configured and "
+                    "therefore must provide the `backend` argument."
+                )
+        elif not isinstance(backend, str):
+            raise TypeError(
+                "backend must be a dotted import path string (got %r)." % backend
+            )
+        else:
+            backend = auth.load_backend(backend)
+        if hasattr(backend, "with_perm"):
+            return backend.with_perm(
+                perm,
+                is_active=is_active,
+                include_superusers=include_superusers,
+                obj=obj,
+            )
+        return self.none()
+
+
 class User(AbstractBaseUser, PermissionsMixin):
-    """Cusomtized version of the default `AbstractUser` from Django.
+    """Cusomtized version of the default `AbstractUser` from Django."""
 
-    """
-    first_name = models.CharField(_('first name'), max_length=100, blank=True)
-    last_name = models.CharField(_('last name'), max_length=100, blank=True)
-    email = models.EmailField(_('email address'), blank=True, unique=True)
+    first_name = models.CharField(_("first name"), max_length=100, blank=True)
+    last_name = models.CharField(_("last name"), max_length=100, blank=True)
+    email = models.EmailField(_("email address"), blank=True, unique=True)
     is_staff = models.BooleanField(
-        _('staff status'), default=False,
-        help_text=_('Designates whether the user can log into this admin '
-                    'site.'))
+        _("staff status"),
+        default=False,
+        help_text=_("Designates whether the user can log into this admin " "site."),
+    )
     is_active = models.BooleanField(
-        _('active'), default=True,
-        help_text=_('Designates whether this user should be treated as '
-                    'active. Unselect this instead of deleting accounts.'))
-    date_joined = models.DateTimeField(_('date joined'), default=timezone.now)
+        _("active"),
+        default=True,
+        help_text=_(
+            "Designates whether this user should be treated as "
+            "active. Unselect this instead of deleting accounts."
+        ),
+    )
+    date_joined = models.DateTimeField(_("date joined"), default=timezone.now)
 
     objects = UserManager()
 
-    USERNAME_FIELD = 'email'
+    USERNAME_FIELD = "email"
     REQUIRED_FIELDS = []
 
     class Meta:
-        verbose_name = _('user')
-        verbose_name_plural = _('users')
+        verbose_name = _("user")
+        verbose_name_plural = _("users")
 
     def get_full_name(self):
         """
         Returns the first_name plus the last_name, with a space in between.
         """
-        full_name = '%s %s' % (self.first_name, self.last_name)
+        full_name = "%s %s" % (self.first_name, self.last_name)
         return full_name.strip()
 
     def get_short_name(self):

sandbox/sandbox/settings.py

@@ -45,8 +45,7 @@
     'wagtail.search',
     'wagtail.admin',
     'wagtail',
-    # 'wagtail_modeladmin',          # if Wagtail >=5.1; Don't repeat if it's there already
-    'wagtail.contrib.modeladmin',    # if Wagtail <5.1;  Don't repeat if it's there already
+    'wagtail_modeladmin',
     'wagtail.contrib.styleguide',
 
     'modelcluster',
@@ -130,8 +129,6 @@
 
 USE_I18N = True
 
-USE_L10N = True
-
 USE_TZ = True
 
 

sandbox/sandbox/urls.py

@@ -1,17 +1,17 @@
 import debug_toolbar
 from django.conf import settings
 from django.contrib import admin
-from django.urls import include, re_path
+from django.urls import include, path, re_path
 
 from wagtail import urls as wagtail_urls
 from wagtail.admin import urls as wagtailadmin_urls
 from wagtail.documents import urls as wagtaildocs_urls
 
 urlpatterns = [
     re_path(r'^admin/', admin.site.urls),
-    re_path(r'^cms/', include(wagtailadmin_urls)),
-    re_path(r'^documents/', include(wagtaildocs_urls)),
-    re_path(r'', include(wagtail_urls)),
+    path('cms/', include(wagtailadmin_urls)),
+    path('documents/', include(wagtaildocs_urls)),
+    path('', include(wagtail_urls)),
 ]
 
 
@@ -24,5 +24,5 @@
     urlpatterns += static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)
 
     urlpatterns = [
-        re_path(r'^__debug__/', include(debug_toolbar.urls)),
+        path('__debug__/', include(debug_toolbar.urls)),
     ] + urlpatterns

setup.py

@@ -3,28 +3,29 @@
 from setuptools import find_packages, setup
 
 install_requires = [
-    "Django>=3.2",
-    "Wagtail>=4.1",
-    "django-otp>=0.8.1",
-    "six>=1.14.0",
-    "qrcode>=6.1",
+    "Django>=4.2",
+    "Wagtail>=6.3",
+    "django-otp>=1.6.1",
+    "six>=1.17.0",
+    "qrcode>=8.2",
 ]
 
 docs_require = [
-    "sphinx>=1.4.1",
-    "sphinx_rtd_theme>=0.4.3",
+    "sphinx>=8.2.3",
+    "sphinx_rtd_theme>=3.0.2",
 ]
 
 tests_require = [
-    "coverage==5.5",
-    "pytest==7.2.2",
-    "pytest-cov==2.12.1",
-    "pytest-django==4.4.0",
+    "coverage==7.10.1",
+    "pytest==8.4.1",
+    "pytest-cov==6.2.1",
+    "pytest-django==4.11.1",
     # Linting
-    "flake8==3.9.2",  # 3.7.9
-    "isort==5.12.0",
-    "flake8-blind-except==0.2.0",
-    "flake8-debugger==4.0.0",
+    "flake8==7.3.0",
+    "isort==6.0.1",
+    "flake8-blind-except==0.2.1",
+    "flake8-debugger==4.1.2",
+    "wagtail-modeladmin==2.2.0"
 ]
 
 with open("README.rst") as fh:
@@ -46,7 +47,7 @@
         "docs": docs_require,
         "test": tests_require,
     },
-    python_requires=">=3.8",
+    python_requires=">=3.10",
     use_scm_version=True,
     entry_points={},
     package_dir={"": "src"},
@@ -57,19 +58,19 @@
         "Development Status :: 4 - Beta",
         "Environment :: Web Environment",
         "Framework :: Django",
-        "Framework :: Django :: 3.2",
-        "Framework :: Django :: 4.1",
         "Framework :: Django :: 4.2",
+        "Framework :: Django :: 5.1",
+        "Framework :: Django :: 5.2",
         "Framework :: Wagtail",
-        "Framework :: Wagtail :: 2",
-        "Framework :: Wagtail :: 3",
-        "Framework :: Wagtail :: 4",
+        "Framework :: Wagtail :: 6",
+        "Framework :: Wagtail :: 7",
         "License :: OSI Approved :: MIT License",
         "Programming Language :: Python",
-        "Programming Language :: Python :: 3.8",
-        "Programming Language :: Python :: 3.9",
         "Programming Language :: Python :: 3.10",
         "Programming Language :: Python :: 3.11",
+        "Programming Language :: Python :: 3.12",
+        "Programming Language :: Python :: 3.13",
+        "Programming Language :: Python :: 3.14",
     ],
     zip_safe=False,
 )

src/wagtail_2fa/templates/wagtail_2fa/device_list.html

@@ -46,7 +46,10 @@ <h2>
 
         {# Users can only add devices to their own account #}
         {% if user_id == request.user.id %}
-            <a href="{% url 'wagtail_2fa_device_new' %}" class="button bicolor icon icon-plus">{% trans 'New device' %}</a>
+            <a href="{% url 'wagtail_2fa_device_new' %}" class="button bicolor button--icon">
+                <span class="icon-wrapper"><svg class="icon icon-plus icon" aria-hidden="true"><use href="#icon-plus"></use></svg></span>
+                {% trans 'New device' %}
+            </a>
         {% endif %}
     </div>
 {% endblock %}