From 4618e0403cd6db0a6607096be15603fd891fbcbf Mon Sep 17 00:00:00 2001 From: Tyler Date: Thu, 19 Feb 2026 20:39:35 -0500 Subject: [PATCH 1/2] [feature] Assign member id automatically on planets - Assign member ID when the target is a planet channel, removing the need for it in the request body --- Valour/Server/Api/Dynamic/MessageApi.cs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Valour/Server/Api/Dynamic/MessageApi.cs b/Valour/Server/Api/Dynamic/MessageApi.cs index 183a3d439..f2453b418 100644 --- a/Valour/Server/Api/Dynamic/MessageApi.cs +++ b/Valour/Server/Api/Dynamic/MessageApi.cs @@ -64,6 +64,8 @@ public static async Task PostMessageRouteAsync( var member = await memberService.GetCurrentAsync(channel.PlanetId.Value); if (member is null) return ValourResult.Forbid("You are not a member of the planet this channel belongs to"); + + message.AuthorMemberId = member.UserId; // NOTE: We don't have to check View permission because lacking view will // cause every other permission check to fail From f8fd7c2039280b72e5ac2553c17792b5bad4c192 Mon Sep 17 00:00:00 2001 From: Tyler Date: Thu, 19 Feb 2026 20:44:31 -0500 Subject: [PATCH 2/2] [bugfix] Fix incorrect field... RTFM --- Valour/Server/Api/Dynamic/MessageApi.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Valour/Server/Api/Dynamic/MessageApi.cs b/Valour/Server/Api/Dynamic/MessageApi.cs index f2453b418..cec29448e 100644 --- a/Valour/Server/Api/Dynamic/MessageApi.cs +++ b/Valour/Server/Api/Dynamic/MessageApi.cs @@ -65,7 +65,7 @@ public static async Task PostMessageRouteAsync( if (member is null) return ValourResult.Forbid("You are not a member of the planet this channel belongs to"); - message.AuthorMemberId = member.UserId; + message.AuthorMemberId = member.Id; // NOTE: We don't have to check View permission because lacking view will // cause every other permission check to fail