Skip to content

Commit

Permalink
Add logging to Role command
Browse files Browse the repository at this point in the history
Resolves #87
  • Loading branch information
super-cooper committed Nov 24, 2024
1 parent 33c66f4 commit 91bf6f9
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions memebot/commands/role.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import discord

from memebot import log
from memebot.lib import exception


Expand Down Expand Up @@ -115,6 +116,9 @@ async def create(interaction: discord.Interaction, role_name: str) -> None:
new_role = await guild.create_role(
name=target_name, mentionable=True, reason=get_reason(author.name)
)
log.interaction(
interaction, f"Created new role @{target_name} for {author.name}"
)
except discord.Forbidden:
raise RolePermissionError("create", target_name)
except discord.HTTPException:
Expand All @@ -141,6 +145,9 @@ async def delete(

try:
await target_role.delete(reason=get_reason(interaction.user.name))
log.interaction(
interaction, f"Deleted role @{target_role.name} for {interaction.user.name}"
)
except discord.Forbidden:
raise RolePermissionError("delete", target_role.name)
except discord.HTTPException:
Expand Down Expand Up @@ -169,6 +176,7 @@ async def join(
)
try:
await author.add_roles(target_role, reason=get_reason(author.name))
log.interaction(interaction, f"Added {author.name} to role @{target_role.name}")
except discord.Forbidden:
raise RolePermissionError("join", target_role.name)
except discord.HTTPException:
Expand Down Expand Up @@ -200,6 +208,9 @@ async def leave(

try:
await author.remove_roles(target_role, reason=get_reason(author.name))
log.interaction(
interaction, f"Removed {author.name} from role @{target_role.name}"
)
except discord.Forbidden:
raise RolePermissionError("leave", target_role.name)
except discord.HTTPException:
Expand Down Expand Up @@ -238,6 +249,10 @@ async def role_list(
member_names.sort()
member_names.insert(0, f"Members of `@{target.name}`:")

log.interaction(
interaction,
f"Listed members of @{target.name} for {interaction.user.name}",
)
await interaction.response.send_message("\n- ".join(member_names))
else:
bot_user = interaction.client.user
Expand All @@ -261,4 +276,5 @@ async def role_list(
0,
f"Roles {usr_msg}managed through `/role` command:",
)
log.interaction(interaction, f"Listed roles for {interaction.user.name}")
await interaction.response.send_message("\n- ".join(roles))

0 comments on commit 91bf6f9

Please sign in to comment.