bot icon indicating copy to clipboard operation
bot copied to clipboard

Unnecessary error logging when cleanbanning a banned user

Open sentry[bot] opened this issue 3 years ago • 2 comments

Sentry Issue: BOT-2ZT

Failed to apply ban to user xxxxxxxxxxxxx

See https://github.com/python-discord/bot/blob/f9fff6eaae409c7ba12ea6f95873aa0fd0340554/bot/exts/moderation/infraction/infractions.py#L125

I think this happens when a user is cleanbanned after being banned. We can probably just remove this log.error as apply_ban already does it's own logging.

Opened by wookie184

sentry[bot] avatar May 02 '22 19:05 sentry[bot]

should probably remove the ctx.send as well as apply_ban will already send a more specific reason

wookie184 avatar May 07 '22 15:05 wookie184

I can take on this issue if needed

JuaniBattiston avatar Aug 05 '22 15:08 JuaniBattiston

I can take on this issue if needed

Sorry for not getting back to you, that would be appreciated, yes!

wookie184 avatar Aug 17 '22 10:08 wookie184

Great, I'll work on it in a few days then.

JuaniBattiston avatar Aug 21 '22 19:08 JuaniBattiston