Add tags to patches in dummy data and fix tag display in history

Author: JelteF

pgcommitfest/commitfest/fixtures/commitfest_data.json

@@ -92,6 +92,96 @@
         "version": "18"
     }
 },
+{
+    "model": "commitfest.tag",
+    "pk": 1,
+    "fields": {
+        "name": "bugfix",
+        "color": "#a51d2d",
+        "description": "Fixes a bug"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 2,
+    "fields": {
+        "name": "backport",
+        "color": "#1a5fb4",
+        "description": "Once merged should be backported to old branches"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 3,
+    "fields": {
+        "name": "missing-tests",
+        "color": "#c66424",
+        "description": "Author should add tests"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 4,
+    "fields": {
+        "name": "missing-docs",
+        "color": "#c66424",
+        "description": "Author should add documentation"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 5,
+    "fields": {
+        "name": "missing-benchmarks",
+        "color": "#c66424",
+        "description": "Author should do additional benchmarks"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 6,
+    "fields": {
+        "name": "help-user-testing",
+        "color": "#07732e",
+        "description": "Reviewers are requested to try out the patch and provide user feedback on behaviour UX/UI"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 7,
+    "fields": {
+        "name": "help-bikeshedding",
+        "color": "#07732e",
+        "description": "Reviewers are requested to propose or vote on stylistic changes like a user facing function name"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 8,
+    "fields": {
+        "name": "help-docs",
+        "color": "#07732e",
+        "description": "Reviewers are requested to help review or write documentation"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 9,
+    "fields": {
+        "name": "help-benchmarks",
+        "color": "#07732e",
+        "description": "Reviewers are requested to help discuss or do benchmarks and discuss performance impact"
+    }
+},
+{
+    "model": "commitfest.tag",
+    "pk": 10,
+    "fields": {
+        "name": "good-first-review",
+        "color": "#613583",
+        "description": "An easy to review patch for a new reviewer"
+    }
+},
 {
     "model": "commitfest.patch",
     "pk": 1,
@@ -103,8 +193,15 @@
         "targetversion": null,
         "committer": null,
         "created": "2025-01-26T10:48:31.579",
-        "modified": "2025-01-26T10:53:20.498",
+        "modified": "2025-06-15T21:26:10.549",
         "lastmail": "2025-01-20T06:53:39",
+        "tags": [
+            2,
+            1,
+            9,
+            6,
+            4
+        ],
         "authors": [
             1
         ],
@@ -126,8 +223,13 @@
         "targetversion": null,
         "committer": null,
         "created": "2025-01-26T10:51:17.305",
-        "modified": "2025-01-26T10:51:19.631",
+        "modified": "2025-06-15T21:26:24.284",
         "lastmail": "2025-01-20T14:20:10",
+        "tags": [
+            10,
+            7,
+            6
+        ],
         "authors": [],
         "reviewers": [],
         "subscribers": [],
@@ -149,6 +251,7 @@
         "created": "2025-01-26T11:02:07.467",
         "modified": "2025-01-26T11:02:10.911",
         "lastmail": "2025-01-20T13:26:55",
+        "tags": [],
         "authors": [],
         "reviewers": [],
         "subscribers": [],
@@ -168,8 +271,11 @@
         "targetversion": null,
         "committer": null,
         "created": "2025-01-31T13:30:19.744",
-        "modified": "2025-01-31T13:30:21.305",
+        "modified": "2025-06-15T21:27:56.667",
         "lastmail": "2025-01-20T12:44:40",
+        "tags": [
+            1
+        ],
         "authors": [],
         "reviewers": [],
         "subscribers": [],
@@ -191,6 +297,7 @@
         "created": "2025-02-16T21:59:04.131",
         "modified": "2025-02-16T22:03:24.902",
         "lastmail": "2025-01-20T14:01:53",
+        "tags": [],
         "authors": [],
         "reviewers": [],
         "subscribers": [],
@@ -212,6 +319,7 @@
         "created": "2025-02-16T22:03:58.476",
         "modified": "2025-02-16T22:04:23.180",
         "lastmail": "2025-01-19T23:55:17",
+        "tags": [],
         "authors": [],
         "reviewers": [],
         "subscribers": [],
@@ -233,6 +341,7 @@
         "created": "2025-03-01T22:27:53.214",
         "modified": "2025-03-01T22:27:53.221",
         "lastmail": "2025-01-18T07:14:02",
+        "tags": [],
         "authors": [],
         "reviewers": [],
         "subscribers": [],
@@ -254,6 +363,7 @@
         "created": "2025-02-01T00:00:00",
         "modified": "2025-02-01T00:00:00",
         "lastmail": "2025-02-01T00:00:00",
+        "tags": [],
         "authors": [
             3,
             6
@@ -576,6 +686,39 @@
         "what": "Attached mail thread example@message-31"
     }
 },
+{
+    "model": "commitfest.patchhistory",
+    "pk": 20,
+    "fields": {
+        "patch": 1,
+        "date": "2025-06-15T21:26:10.546",
+        "by": 1,
+        "by_cfbot": false,
+        "what": "Changed tags to backport, bugfix, help-benchmarks, help-user-testing, missing-docs"
+    }
+},
+{
+    "model": "commitfest.patchhistory",
+    "pk": 21,
+    "fields": {
+        "patch": 2,
+        "date": "2025-06-15T21:26:24.282",
+        "by": 1,
+        "by_cfbot": false,
+        "what": "Changed tags to good-first-review, help-bikeshedding, help-user-testing"
+    }
+},
+{
+    "model": "commitfest.patchhistory",
+    "pk": 22,
+    "fields": {
+        "patch": 4,
+        "date": "2025-06-15T21:27:56.664",
+        "by": 1,
+        "by_cfbot": false,
+        "what": "Changed tags to bugfix"
+    }
+},
 {
     "model": "commitfest.mailthread",
     "pk": 1,

pgcommitfest/commitfest/views.py

@@ -774,10 +774,14 @@ def patchform(request, patchid):
 
             # Track all changes
             for field, values in r.diff.items():
+                if field == "tags":
+                    value = ", ".join(v.name for v in values[1])
+                else:
+                    value = values[1]
                 PatchHistory(
                     patch=patch,
                     by=request.user,
-                    what="Changed %s to %s" % (field, values[1]),
+                    what="Changed %s to %s" % (field, value),
                 ).save_and_notify(
                     prevcommitter=prevcommitter,
                     prevreviewers=prevreviewers,