-
-
Notifications
You must be signed in to change notification settings - Fork 750
ID not displayed in dm_relay when attachment sent #1137
Copy link
Copy link
Closed
Labels
a: moderationRelated to community moderation functionality: (moderation, defcon, verification)Related to community moderation functionality: (moderation, defcon, verification)p: 2 - normalNormal PriorityNormal Priorityt: bugSomething isn't workingSomething isn't working
Metadata
Metadata
Assignees
Labels
a: moderationRelated to community moderation functionality: (moderation, defcon, verification)Related to community moderation functionality: (moderation, defcon, verification)p: 2 - normalNormal PriorityNormal Priorityt: bugSomething isn't workingSomething isn't working
Type
Fields
Give feedbackNo fields configured for issues without a type.
Currently the dm-relay message username will not include the user ID if only an attachment is sent, so it looks like this:


Whereas it should look like this:
This is because it uses
bot.utils.messages.send_attachments, which by default just sends the message with the ID. Relevant line indm_relaycog hereSimple solution here would be to add a
usernamekeyword argument tosend_attachments. I would be happy to make this change if this all sounds good.