#insiders-info: Fix applying roles when user has a managed role

FloatingMilkshake · FloatingMilkshake · commit 227f45475f4e · 2025-01-03T13:49:12.000-05:00
diff --git a/Events/InteractionEvents.cs b/Events/InteractionEvents.cs
@@ -283,7 +283,8 @@ await LogChannelHelper.LogDeletedMessagesAsync(
                     }
                 }
                 
-                await member.ReplaceRolesAsync(memberRoles, "Applying Insider roles chosen in #insiders-info");
+                // Apply roles
+                await member.ModifyAsync(x => x.Roles = memberRoles);
                 
                 await e.Interaction.CreateFollowupMessageAsync(new DiscordFollowupMessageBuilder().WithContent($"{cfgjson.Emoji.Success} Your Insider roles have been updated!").AsEphemeral(true));
             }

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,8 @@ await LogChannelHelper.LogDeletedMessagesAsync(`
`283`	`283`	`}`
`284`	`284`	`}`
`285`	`285`
`286`		`- await member.ReplaceRolesAsync(memberRoles, "Applying Insider roles chosen in #insiders-info");`
	`286`	`+ // Apply roles`
	`287`	`+ await member.ModifyAsync(x => x.Roles = memberRoles);`
`287`	`288`
`288`	`289`	`await e.Interaction.CreateFollowupMessageAsync(new DiscordFollowupMessageBuilder().WithContent($"{cfgjson.Emoji.Success} Your Insider roles have been updated!").AsEphemeral(true));`
`289`	`290`	`}`