Skip to content

Mentionables

Mateusz Brawański requested to merge github/fork/Lachee/patch/mentionables into master

Created by: Lachee

Summary

Added support for Mentionables

Details

Adds support for Discord's new mentionables. https://discordapp.com/developers/docs/resources/channel#allowed-mentions-object

Changes proposed

  • Created new DiscordMentions object for internals
  • Created new IMention. UserMention, RoleMention and EveryoneMention extend this.
  • Updated CreateMessage and UploadFiles internals and usages to support IEnumerable<IMention>

Notes

Tested and it works. There is a edge case to account for: If a user says they want to whitelist a specific user AND allow all user mentions. The discord API will throw a bad request if this is sent. I am currently just ignoring the specific whitelist and assuming the user actually wanted the allow all user mentions. Is this the best case?

Merge request reports