Add unwarn modlog (with option on dashbord)

[ColinLeDev]

Starting from https://discord.com/channels/166207328570441728/1316169654502293514

• Added two options (db+dashboard)
  UnwarnSendToModlog: In the name
  UnwarnIncludeWarnReason: Include the dbStored (not the modlog message) original warn reason
• Added a ModlogAction (You should choose color & emote)
  MAUnwarned = ModlogAction{Prefix: "Warn removed", Emoji: "🧽", Color: 0xfca253}

23/dec/2024 : I did found an old suggestion

[ashishjh-bst]

It is isn't still clear to me what this PR actually does and what is the purpose of it. Please showcase demo usage along with screenshots.

[ColinLeDev]

It is isn't still clear to me what this PR actually does and what is the purpose of it. Please showcase demo usage along with screenshots.

You can see a demo here https://sh.col1n.fr/YAG-PR1795-demo.html
(modlog is the channel) (I removed the actual deletion for the demo)

[lemm-e]

I don't understand the purpose of displaying warn reason in the deletewarning entry. We don't do this with mutes or bans, so why do it here?

[ColinLeDev]

At first, I wasn't going to add the original warn reason, but I was already getting the warn entry, and it can be useful for admins to quickly have the original reason without looking in the modlogs (which don't display the warnID)

So if you delete a warning, there is no way for the admin to have the original reason
Mutes, ban, kick, to are discord Audit-logged if I remember correctly

[lemm-e]

If you delete a warning, you do so because you don't need it anymore. So why would you wanna have the original reason? Imo consistency amongst all commands should be favored over the little bit of effort it would take to look at the mod-logs to find the original reason if it is ever needed at all

[ColinLeDev]

Fair point, I can remove that part (but modlog the delwarn is consistent)
Anyone against the removal ?

To complete Soggy's response : ID are not related to server, but are overall yag (> 150M currently)

Again, only thing not Audit-logged

[SoggySaussages]

I am against removal. If you’re unmuted, you know there’s only one mute they can be talking about. If you’re unbanned, there could only be one ban you’re talking about. If you delete a warning, there could be multiple warnings and you are only removing one. If the only identifier is a warning ID, it will be literally impossible to find which warning it is referring to without manually counting every warning ever issued on your server.