eris icon indicating copy to clipboard operation
eris copied to clipboard

Published 20 hours ago • verified publisher icon abalabahaha

Attachment type

Open coolcalcacol opened this issue 2 years ago • 6 comments

coolcalcacol avatar Apr 12 '22 09:04 coolcalcacol

Is this solely for the attachment type? There is also an attachments map in command interaction.

Linker-123 avatar Apr 12 '22 10:04 Linker-123

@Linker-123 Is that better?

coolcalcacol avatar Apr 12 '22 11:04 coolcalcacol

Add types for attachment class

Linker-123 avatar Apr 12 '22 12:04 Linker-123

what was the conclusion with the attachments class vs just using objects?

coolcalcacol avatar May 20 '22 13:05 coolcalcacol

The interface pre-dates this pr. Before I headed away I was considering merging it but that would mean processing the message to create attachment classes for each attachment

coolcalcacol avatar Jun 30 '22 10:06 coolcalcacol

That sounds like the best idea we've got it now, so why not use it

it's either that or rename one of them

DonovanDMC avatar Jun 30 '22 10:06 DonovanDMC