Skip to content

Commit 4410791

Browse files
committed
add logging for if role syncing does change anything
1 parent ba4fff4 commit 4410791

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

TrickFireDiscordBot/Services/RoleSyncer.cs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -165,16 +165,19 @@ public async Task SyncRoles(DiscordMember member, bool dryRun = true)
165165
));
166166

167167
string logString = string.Join("`, `", newRoles.Select(role => role.Name));
168+
bool hasChanged = newRoles.SetEquals(member.Roles);
168169
logger.LogDebug(
169-
"(dry run: `{}`) Roles for user `{}` (`{}`): `{}`",
170+
"(dry run: `{}`, changed?: `{}`) Roles for user `{}` (`{}`): `{}`",
170171
dryRun,
172+
hasChanged,
171173
member.Username,
172174
member.DisplayName,
173175
logString
174176
);
175177
messageLogger.LogDebug(
176-
"(dry run: `{}`) Roles for user `{}` (`{}`): `{}`",
178+
"(dry run: `{}`, changed?: `{}`) Roles for user `{}` (`{}`): `{}`",
177179
dryRun,
180+
hasChanged,
178181
member.Username,
179182
member.DisplayName,
180183
logString

0 commit comments

Comments
 (0)