links in admin logs

jerimiah-smiggins · Oct 13, 2024 · e5419f1 · e5419f1
1 parent f810a4b
commit e5419f1
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/smiggins/backend/api/admin.py b/smiggins/backend/api/admin.py
@@ -651,7 +651,7 @@ def logs(request) -> tuple | dict:
                 "target": i.uname_for or (i.u_for and i.u_for.username),
                 "info": i.info,
                 "timestamp": i.timestamp
-            } for i in AdminLog.objects.all()]
+            } for i in AdminLog.objects.all()][::-1]
         }
 
     return 400, {

diff --git a/smiggins/lang/en-US.json b/smiggins/lang/en-US.json
@@ -229,8 +229,8 @@
         "action": "Action",
         "who": "Who",
         "more_info": "More Info",
-        "who_format": "Done by @%1, at @%2",
-        "who_format_single": "Done by @%1"
+        "who_format": "Done by %1, at %2",
+        "who_format_single": "Done by %1"
       }
     },
 

diff --git a/smiggins/static/js/admin.js b/smiggins/static/js/admin.js
@@ -185,7 +185,7 @@ testMask(Mask.ReadLogs) && dom("debug-button").addEventListener("click", functio
                     output += `<tr>
               <td class="nowrap">${timeSince(+line.timestamp)}</td>
               <td class="nowrap">${line.type}</td>
-              <td>${(lang.admin.logs[line.target ? "who_format" : "who_format_single"].replaceAll("%1", line.by).replaceAll("%2", line.target)).replaceAll(" ", "&nbsp;").replaceAll(",&nbsp;", ", ")}</td>
+              <td>${lang.admin.logs[line.target ? "who_format" : "who_format_single"].replaceAll(" ", "&nbsp;").replaceAll(",&nbsp;", ", ").replaceAll("%1", `<a href="/u/${line.by}">@${line.by}</a>`).replaceAll("%2", `<a href="/u/${line.target}">@${line.target}</a>`)}</td>
               <td>${escapeHTML(line.info)}</td>
             </tr>`;
                 }

diff --git a/smiggins/static/ts/admin.ts b/smiggins/static/ts/admin.ts
@@ -228,7 +228,7 @@ testMask(Mask.ReadLogs) && dom("debug-button").addEventListener("click", functio
             output += `<tr>
               <td class="nowrap">${timeSince(+line.timestamp)}</td>
               <td class="nowrap">${line.type}</td>
-              <td>${(lang.admin.logs[line.target ? "who_format" : "who_format_single"].replaceAll("%1", line.by).replaceAll("%2", line.target)).replaceAll(" ", "&nbsp;").replaceAll(",&nbsp;", ", ")}</td>
+              <td>${lang.admin.logs[line.target ? "who_format" : "who_format_single"].replaceAll(" ", "&nbsp;").replaceAll(",&nbsp;", ", ").replaceAll("%1", `<a href="/u/${line.by}">@${line.by}</a>`).replaceAll("%2", `<a href="/u/${line.target}">@${line.target}</a>`)}</td>
               <td>${escapeHTML(line.info)}</td>
             </tr>`;
           } catch(err) {