Merge branch 'miscs'

Author: Jenselme

.github/dependabot.yml

@@ -61,11 +61,9 @@ updates:
       interval: "weekly"
 
   # Enable version updates for Python/Pip - Production
-  - package-ecosystem: "pip"
-    # Look for a `requirements.txt` in the `root` directory
-    # also 'setup.cfg', 'runtime.txt' and 'requirements/*.txt'
+  - package-ecosystem: "uv"
     directory: "/"
     open-pull-requests-limit: 15
     # Every weekday
     schedule:
-      interval: "daily"
+      interval: "weekly"

legadilo/reading/models/article.py

@@ -19,6 +19,7 @@
 import json
 import logging
 import math
+import re
 from collections.abc import Iterable
 from dataclasses import dataclass
 from itertools import chain
@@ -599,6 +600,7 @@ def create_invalid_article(
                 user=user,
                 link=article_link,
                 title=full_sanitize(article_link),
+                slug=slugify(re.sub(r"^https?://", "", article_link)),
                 main_source_type=constants.ArticleSourceType.MANUAL,
                 main_source_title=article_domain,
             )
@@ -834,8 +836,7 @@ def __str__(self):
         )
 
     def save(self, *args, **kwargs):
-        if not self.slug:
-            self.slug = slugify(self.title)
+        self.slug = self.slug or slugify(self.title) or str(_("no-slug"))
 
         return super().save(*args, **kwargs)
 

legadilo/reading/tests/test_models/test_article.py

@@ -1342,12 +1342,19 @@ def test_create_invalid_article(self, user, django_assert_num_queries):
         assert created
         assert article.link == link
         assert article.title == link
+        assert article.slug == "toto-com"
         assert article.updated_at is None
         assert article.main_source_type == constants.ArticleSourceType.MANUAL
         assert article.main_source_title == "toto.com"
         assert list(article.tags.all()) == [tag]
         assert article.article_fetch_errors.count() == 1
 
+    def test_create_article_title_cannot_be_slugified(self, user):
+        article = Article.objects.create(link="https://toto.com/article", title="??", user=user)
+
+        assert article.title == "??"
+        assert article.slug == "no-slug"
+
     def test_create_invalid_article_article_already_saved(self, user, django_assert_num_queries):
         initial_article = ArticleFactory(user=user)
         tag = TagFactory(user=user, title="Test")

legadilo/reading/tests/test_views/test_fetch_article_view.py

@@ -157,7 +157,7 @@ def test_fetch_failure(self, logged_in_sync_client, httpx_mock):
                 message="Failed to fetch the article. Please check that the URL you entered is "
                 "correct, that the article exists and is accessible. "
                 "We added its URL directly. "
-                f'Go <a href="/reading/articles/{article.id}-https-www-example-com-posts-en-1-super-article/'  # noqa: E501
+                f'Go <a href="/reading/articles/{article.id}-www-example-com-posts-en-1-super-article/'  # noqa: E501
                 '">there</a> to access it.',
             )
         ]
@@ -233,7 +233,7 @@ def test_content_too_big(self, logged_in_sync_client, httpx_mock, mocker):
                 level=DEFAULT_LEVELS["WARNING"],
                 message="The article you are trying to fetch is too big and cannot be processed. "
                 "We added its URL directly. "
-                f'Go <a href="/reading/articles/{article.id}-https-www-example-com-posts-en-1-super-article/"'  # noqa: E501
+                f'Go <a href="/reading/articles/{article.id}-www-example-com-posts-en-1-super-article/"'  # noqa: E501
                 ">there</a> to access it.",
             )
         ]

legadilo/users/admin.py

@@ -64,9 +64,9 @@ class UserAdmin(auth_admin.UserAdmin):
         ),
         (_("Important dates"), {"fields": ("last_login", "date_joined")}),
     )
-    list_display = ["email", "name", "is_superuser"]
+    list_display = ["email", "date_joined", "last_login", "is_active", "is_superuser"]
     search_fields = ["name", "email_deterministic"]
-    ordering = ["id"]
+    ordering = ["last_login"]
     add_fieldsets = (
         (
             None,

pyproject.toml

@@ -64,7 +64,7 @@ dev = [
     "pre-commit >= 4.0.1", # https://github.com/pre-commit/pre-commit
     # Django
     "factory-boy >= 3.3.1",  # https://github.com/FactoryBoy/factory_boy
-    "django-debug-toolbar >= 4.4.6",  # https://github.com/jazzband/django-debug-toolbar
+    "django-debug-toolbar >= 4.4.6, < 5.1.0",  # https://github.com/jazzband/django-debug-toolbar
     "django-coverage-plugin >= 3.1.0",  # https://github.com/nedbat/django_coverage_plugin
     "pytest-django >= 4.9.0",  # https://github.com/pytest-dev/pytest-django
     "django-browser-reload >= 1.17.0",  # https://pypi.org/project/django-browser-reload/