feat(changes): expose RSS feeds for all change views

This allows users to follow any changes search, not only these few
predefined views.

Fixes #7518

Author: nijel

docs/api.rst

@@ -3291,6 +3291,34 @@ RSS feeds
 
 Changes in translations are exported in RSS feeds.
 
+Filtered RSS feeds are available from the changes browser. These accept the same
+filters as the changes page, for example ``action``, ``user``, ``exclude_user``,
+and ``period``.
+
+.. http:get:: /changes/rss/
+
+    Retrieves RSS feed with recent changes matching changes browsing filters.
+
+.. http:get:: /changes/rss/(string:project)/(string:component)/(string:language)/
+
+    Retrieves RSS feed with recent changes matching changes browsing filters in a
+    translation.
+
+.. http:get:: /changes/rss/(string:project)/(string:component)/
+
+    Retrieves RSS feed with recent changes matching changes browsing filters in a
+    component.
+
+.. http:get:: /changes/rss/(string:project)/-/(string:language)/
+
+    Retrieves RSS feed with recent changes matching changes browsing filters in a
+    project language.
+
+.. http:get:: /changes/rss/-/-/(string:language)/
+
+    Retrieves RSS feed with recent changes matching changes browsing filters in a
+    language.
+
 .. http:get:: /exports/rss/(string:project)/(string:component)/(string:language)/
 
     Retrieves RSS feed with recent changes for a translation.

docs/changes.rst

@@ -26,6 +26,7 @@ Weblate 2026.5
 * The translating page now separates screenshots from string information, collapses rarely used string details, and groups glossary and screenshot actions more consistently.
 * Project access management now paginates users and better explains site-wide automatic team assignments.
 * Documented Gettext-style :ref:`plural-formula` syntax and linked to the upstream GNU gettext references.
+* :ref:`RSS feeds <rss>` can now use the same filters as the changes browsing page.
 
 .. rubric:: Bug fixes
 

weblate/templates/trans/change_list.html

@@ -38,7 +38,7 @@
            class="btn btn-link">{% icon "download.svg" %}</a>
       {% endif %}
     {% endif %}
-    <a href="{{ changes_rss }}"
+    <a href="{{ changes_rss }}{% if query_string %}?{{ query_string }}{% endif %}"
        title="{% translate "Follow using RSS" %}"
        class="btn btn-link">{% icon "rss.svg" %}</a>
   </div>

weblate/trans/tests/test_changes.py

@@ -11,6 +11,7 @@
 from django.utils import timezone
 from django.utils.http import urlencode
 
+from weblate.trans.actions import ActionEvents
 from weblate.trans.feeds import TranslationChangesFeed
 from weblate.trans.models import Change, Unit
 from weblate.trans.tests.test_views import FixtureTestCase, ViewTestCase
@@ -61,10 +62,27 @@ def test_translation_feed_queries_do_not_scale_with_change_count(self) -> None:
 
 
 class ChangesTest(ViewTestCase):
+    def assert_rss_response(self, response) -> None:
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response["Content-Type"], "application/rss+xml; charset=utf-8")
+        self.assertContains(response, "<rss")
+
+    def add_filtered_rss_changes(self) -> None:
+        Change.objects.all().delete()
+        self.change_unit("Initial RSS target\n", user=self.user)
+        Change.objects.all().delete()
+        self.change_unit("Current user RSS target\n", user=self.user)
+        self.change_unit("Another user RSS target\n", user=self.anotheruser)
+
     def test_basic(self) -> None:
         response = self.client.get(reverse("changes"))
         self.assertContains(response, "Resource update")
 
+    def test_basic_rss(self) -> None:
+        response = self.client.get(reverse("changes-rss"))
+        self.assert_rss_response(response)
+        self.assertContains(response, "Resource update")
+
     def test_basic_csv_denied(self) -> None:
         response = self.client.get(reverse("changes-csv"))
         self.assertEqual(response.status_code, 403)
@@ -113,6 +131,13 @@ def test_user(self) -> None:
         )
         self.assertNotContains(response, f'title="{self.user.full_name}"')
 
+    def test_user_rss(self) -> None:
+        self.add_filtered_rss_changes()
+        response = self.client.get(reverse("changes-rss"), {"user": self.user.username})
+        self.assert_rss_response(response)
+        self.assertContains(response, "testuser")
+        self.assertNotContains(response, "Jane Doe")
+
     def test_exclude_user(self) -> None:
         self.edit_unit("Hello, world!\n", "Nazdar svete!\n")
         response = self.client.get(reverse("changes"))
@@ -130,13 +155,76 @@ def test_exclude_user(self) -> None:
         )
         self.assertContains(response, f'title="{self.user.full_name}"')
 
+    def test_exclude_user_rss(self) -> None:
+        self.add_filtered_rss_changes()
+        response = self.client.get(
+            reverse("changes-rss"), {"exclude_user": self.user.username}
+        )
+        self.assert_rss_response(response)
+        self.assertNotContains(response, "Weblate Test")
+        self.assertContains(response, "Jane Doe")
+
+    def test_action_rss(self) -> None:
+        Change.objects.all().delete()
+        self.change_unit("Initial RSS target\n", user=self.user)
+        Change.objects.all().delete()
+        self.change_unit("Current user RSS target\n", user=self.user)
+        response = self.client.get(
+            reverse("changes-rss"), {"action": ActionEvents.CHANGE}
+        )
+        self.assert_rss_response(response)
+        self.assertContains(response, "Translation changed")
+        response = self.client.get(
+            reverse("changes-rss"), {"action": ActionEvents.UPDATE}
+        )
+        self.assert_rss_response(response)
+        self.assertNotContains(response, "Translation changed")
+
     def test_daterange(self) -> None:
         end = timezone.now()
         start = end - timedelta(days=1)
         period = f"{start.strftime('%m/%d/%Y')} - {end.strftime('%m/%d/%Y')}"
         response = self.client.get(reverse("changes"), {"period": period})
         self.assertContains(response, "Resource update")
 
+    def test_daterange_rss(self) -> None:
+        Change.objects.all().delete()
+        self.change_unit("Initial RSS target\n", user=self.user)
+        Change.objects.all().delete()
+        self.change_unit("Current user RSS target\n", user=self.user)
+        end = timezone.now()
+        start = end - timedelta(days=1)
+        period = f"{start.strftime('%m/%d/%Y')} - {end.strftime('%m/%d/%Y')}"
+        response = self.client.get(reverse("changes-rss"), {"period": period})
+        self.assert_rss_response(response)
+        self.assertContains(response, "Translation changed")
+        response = self.client.get(
+            reverse("changes-rss"), {"period": "01/01/2020 - 01/02/2020"}
+        )
+        self.assert_rss_response(response)
+        self.assertNotContains(response, "Translation changed")
+
+    def test_scoped_rss(self) -> None:
+        Change.objects.all().delete()
+        self.change_unit("Initial RSS target\n", user=self.user)
+        Change.objects.all().delete()
+        unit = self.change_unit("Scoped RSS target\n", user=self.user)
+        paths = (
+            self.project.get_url_path(),
+            self.component.get_url_path(),
+            self.translation.get_url_path(),
+            ["-", "-", self.translation.language.code],
+            [self.project.slug, "-", self.translation.language.code],
+            unit.get_url_path(),
+        )
+        for path in paths:
+            with self.subTest(path=path):
+                response = self.client.get(
+                    reverse("changes-rss", kwargs={"path": path})
+                )
+                self.assert_rss_response(response)
+                self.assertContains(response, "Translation changed")
+
     def test_pagination(self) -> None:
         end = timezone.now()
         start = end - timedelta(days=1)
@@ -149,6 +237,11 @@ def test_pagination(self) -> None:
         )
         self.assertContains(response, "String added in the upload")
 
+    def test_rss_link_keeps_query_string(self) -> None:
+        response = self.client.get(reverse("changes"), {"user": self.user.username})
+        query_string = urlencode({"user": self.user.username})
+        self.assertContains(response, f"{reverse('changes-rss')}?{query_string}")
+
     def test_last_changes_display(self) -> None:
         unit_to_delete = self.get_unit("Orangutan has %d banana")
         unit_to_delete.context = "Orangutan unit context"

weblate/trans/views/changes.py

@@ -7,13 +7,15 @@
 import csv
 from typing import TYPE_CHECKING
 
+from django.conf import settings
 from django.contrib.auth.decorators import login_required
 from django.core.exceptions import PermissionDenied
 from django.http import HttpResponse
 from django.shortcuts import get_object_or_404, redirect
 from django.urls import reverse
 from django.urls.exceptions import NoReverseMatch
-from django.utils.translation import activate, gettext, pgettext
+from django.utils import feedgenerator
+from django.utils.translation import activate, get_language, gettext, pgettext
 from django.views.generic.list import ListView
 
 from weblate.accounts.notifications import NOTIFICATIONS_ACTIONS
@@ -45,6 +47,44 @@ class ChangesView(PathViewMixin, ListView):
         Unit,
     )
 
+    def get_changes_url(self, url_name: str = "changes") -> str:
+        if self.path_object is None:
+            return reverse(url_name)
+        return reverse(url_name, kwargs={"path": self.path_object.get_url_path()})
+
+    def get_filtered_changes_url(self) -> str:
+        url = self.get_changes_url()
+        if self.changes_form.is_valid() and (
+            query_string := self.changes_form.urlencode()
+        ):
+            return f"{url}?{query_string}"
+        return url
+
+    def get_title(self):
+        if isinstance(self.path_object, Unit):
+            return (
+                pgettext(
+                    "Changes of string in a translation", "Changes of string in %s"
+                )
+                % self.path_object
+            )
+        if isinstance(self.path_object, Translation):
+            return (
+                pgettext("Changes in translation", "Changes in %s") % self.path_object
+            )
+        if isinstance(self.path_object, Component):
+            return pgettext("Changes in component", "Changes in %s") % self.path_object
+        if isinstance(self.path_object, Project):
+            return pgettext("Changes in project", "Changes in %s") % self.path_object
+        if isinstance(self.path_object, Language):
+            return pgettext("Changes in language", "Changes in %s") % self.path_object
+        if isinstance(self.path_object, ProjectLanguage):
+            return pgettext("Changes in project", "Changes in %s") % self.path_object
+        if self.path_object is None:
+            return gettext("Changes")
+        msg = f"Unsupported {self.path_object}"
+        raise TypeError(msg)
+
     def get_template_names(self):
         if digest := self.request.GET.get("digest"):
             if digest in {"pending_suggestions", "todo_strings"}:
@@ -56,46 +96,8 @@ def get_context_data(self, **kwargs):
         """Create context for rendering page."""
         context = super().get_context_data(**kwargs)
         context["path_object"] = self.path_object
-
-        if isinstance(self.path_object, Unit):
-            context["title"] = (
-                pgettext(
-                    "Changes of string in a translation", "Changes of string in %s"
-                )
-                % self.path_object
-            )
-        elif isinstance(self.path_object, Translation):
-            context["title"] = (
-                pgettext("Changes in translation", "Changes in %s") % self.path_object
-            )
-        elif isinstance(self.path_object, Component):
-            context["title"] = (
-                pgettext("Changes in component", "Changes in %s") % self.path_object
-            )
-        elif isinstance(self.path_object, Project):
-            context["title"] = (
-                pgettext("Changes in project", "Changes in %s") % self.path_object
-            )
-        elif isinstance(self.path_object, Language):
-            context["title"] = (
-                pgettext("Changes in language", "Changes in %s") % self.path_object
-            )
-        elif isinstance(self.path_object, ProjectLanguage):
-            context["title"] = (
-                pgettext("Changes in project", "Changes in %s") % self.path_object
-            )
-        elif self.path_object is None:
-            context["title"] = gettext("Changes")
-        else:
-            msg = f"Unsupported {self.path_object}"
-            raise TypeError(msg)
-
-        if self.path_object is None:
-            context["changes_rss"] = reverse("rss")
-        else:
-            context["changes_rss"] = reverse(
-                "rss", kwargs={"path": self.path_object.get_url_path()}
-            )
+        context["title"] = self.get_title()
+        context["changes_rss"] = self.get_changes_url("changes-rss")
 
         if self.changes_form.is_valid():
             context["query_string"] = self.changes_form.urlencode()
@@ -126,7 +128,9 @@ def get_request_param(self, request: AuthenticatedHttpRequest, param: str) -> st
             return "-"
         return value
 
-    def get(self, request: AuthenticatedHttpRequest, *args, **kwargs):  # type: ignore[override]
+    def get(  # type: ignore[override]
+        self, request: AuthenticatedHttpRequest, *args, **kwargs
+    ):
         if self.path_object is None and self.request.GET:
             # Handle GET params for filtering prior Weblate 5.0
             path = None
@@ -220,7 +224,9 @@ class ChangesCSVView(ChangesView):
 
     paginate_by = None
 
-    def get(self, request: AuthenticatedHttpRequest, *args, **kwargs):  # type: ignore[override]
+    def get(  # type: ignore[override]
+        self, request: AuthenticatedHttpRequest, *args, **kwargs
+    ):
         object_list = self.get_queryset()[:2000]
 
         if not request.user.has_perm("change.download", self.path_object):
@@ -254,6 +260,65 @@ def get(self, request: AuthenticatedHttpRequest, *args, **kwargs):  # type: igno
         return response
 
 
+class ChangesRSSView(ChangesView):
+    """RSS renderer for changes view."""
+
+    feed_count = 10
+    paginate_by = None
+
+    def get_feed_title(self):
+        if self.path_object is None:
+            # Translators: %s is site title here
+            return gettext("Recent changes on %s") % settings.SITE_TITLE
+        # Translators: %s is translation/project/component/language name
+        return gettext("Recent changes in %s") % self.path_object
+
+    def get_feed_description(self):
+        if self.path_object is None:
+            # Translators: %s is site title here
+            return gettext("All recent changes made using Weblate on %s.") % (
+                settings.SITE_TITLE
+            )
+        # Translators: %s is translation/project/component/language name
+        return (
+            gettext("All recent changes made using Weblate in %s.") % self.path_object
+        )
+
+    def get(  # type: ignore[override]
+        self, request: AuthenticatedHttpRequest, *args, **kwargs
+    ):
+        if self.changes_form.is_valid():
+            object_list = Change.objects.preload_list(
+                list(self.get_queryset()[: self.feed_count])
+            )
+        else:
+            object_list = []
+
+        feed = feedgenerator.Rss201rev2Feed(
+            title=self.get_feed_title(),
+            link=get_site_url(self.get_filtered_changes_url()),
+            description=self.get_feed_description(),
+            language=get_language(),
+            feed_url=get_site_url(request.get_full_path()),
+        )
+
+        for change in object_list:
+            link = get_site_url(change.get_absolute_url())
+            feed.add_item(
+                title=change.get_action_display(),
+                link=link,
+                description=str(change),
+                author_name=change.get_user_display(False),
+                pubdate=change.timestamp,
+                unique_id=link,
+                unique_id_is_permalink=True,
+            )
+
+        response = HttpResponse(content_type=feed.content_type)
+        feed.write(response, "utf-8")
+        return response
+
+
 @login_required
 def show_change(request: AuthenticatedHttpRequest, pk: int):
     change = get_object_or_404(Change, pk=pk)