From 9a2bb696014c3b4ff4d43ec1223dc654c00bb673 Mon Sep 17 00:00:00 2001
From: Jemma Poffinbarger <git@jemsoftware.dev>
Date: Sat, 11 Jan 2025 16:51:05 -0600
Subject: [PATCH] feat: Stop showing who banned a user in the ban message

---
 src/models/settings.js                      | 2 ++
 src/services/juxt-web/routes/admin/admin.js | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/models/settings.js b/src/models/settings.js
index cef9d78..53a64a7 100644
--- a/src/models/settings.js
+++ b/src/models/settings.js
@@ -8,6 +8,8 @@ const SettingsSchema = new Schema({
 		default: 0
 	},
 	ban_lift_date: Date,
+	// TODO: Move bans to their own collection. User settings should be scoped to older the user
+	banned_by: Number,
 	ban_reason: String,
 	profile_comment: {
 		type: String,
diff --git a/src/services/juxt-web/routes/admin/admin.js b/src/services/juxt-web/routes/admin/admin.js
index df8bc7b..44ae89d 100644
--- a/src/services/juxt-web/routes/admin/admin.js
+++ b/src/services/juxt-web/routes/admin/admin.js
@@ -118,7 +118,8 @@ router.post('/accounts/:pid', async (req, res) => {
 	await SETTINGS.findOneAndUpdate({ pid: pid }, {
 		account_status: req.body.account_status,
 		ban_lift_date: req.body.ban_lift_date,
-		ban_reason: `${req.user.username} (${req.pid}): ${req.body.ban_reason}`
+		banned_by: req.pid,
+		ban_reason: req.body.ban_reason
 	});
 
 	res.json({