search

ShimmyMySherbet / DiscordWebhooks

A lightweight discord webhook client with modern message constructors.
MIT License
8 stars 4 forks source link

Added the ability to edit messages. #5

Closed Jdance-Media closed 4 months ago

Jdance-Media commented 1 year ago

Will throw an exception if you try to use the username or avatar_url fields. Still, should be a welcomed addition, seeing as how I desperately need to be able to edit messages easily.

ShimmyMySherbet commented 1 year ago

Thanks for getting onto this, I'd forgotten about that issue. Though I'm not sure if throwing an exception on username or pfp being set. Since while those fields cannot be changed, I feel it could complicate webhook message generation. I.E., requires different methods/flags for generating the initial message and updated message.

ShimmyMySherbet commented 1 year ago

I have since done a major refactoring on this project, and also created a new branch to stage this new feature. If you want, you can amend this PR to move changes to the new message posting implementation in ShimmyMySherbet.DiscordWebhooks.DiscordWebhookService.

ShimmyMySherbet commented 1 year ago

I'm likely to re-write this for the newer version, as I'm also looking at implementing methods to retrieve the message ID of posted messages, and also adding thread support. Though you can merge this into the embed-able file 'LegacyEmbedded.cs'

Jdance-Media commented 1 year ago

I'm awfully busy so I'll get around to merging it into LeagacyEmbedded.cs in a few days.