Inline Replies

[sh30801]

Summary

This issue is for tracking developments related to inline replies. Right now it looks like it will be using message references as well as adding new data to message and activity types.

[sh30801]

Currently waiting on discord/discord-api-docs#2118 to be merged.

[ProfDoof]

discord/discord-api-docs#2118 was merged.

[VelvetToroyashi]

How hard could it be :^)

[ProfDoof]

Famous last words.

Anyway, so to start a conversation about the expected changes.

1. Add the ReferencedMessage Property to the DiscordMessage. Should have type DiscordMessage
2. Update the XML for the MessageReference to reflect the changes in the way that it has been used
3. Add the new message type to the Enum MessageType
4. Add a ReplyAsync method to the DiscordMessage class (I'm going based on the fact that message_id is the only requirement to reply to a message according to the comment thread and this portion of the docs ).
5. Add a ReplyAsync method to the CommandContext class.

Since we don't have an AllowedMentions class as seen here, we should make a boolean on our ReplyAsync methods to specify whether to ping the user or not. Should we default to false on pinging on our side? Or should we leave the default behavior up to Discord?

This is all I can think of right now, and I might have missed some things. Let me know what y'all think.

[VelvetToroyashi]

Well at least on the client side, it defaults to mentioning.

I'd be more than happy to help when it comes to adding the DiscordMessage#ReplyAsync and CommandContext#ReplyAsync.

[VelvetToroyashi]

Also just gonna go on a hunch, given that the message reference only has a few Ids instead of a proper message object, I take it that property would only be a partial-object, and it's up to the end user to actually retrieve the message from REST if they want to get the full object? I don't know the deep inner-workings of the API, so that's my best guess :p

[ProfDoof]

From my (admittedly very basic) understanding referenced_message is a full message object. At least in V8. Not in V6. It doesn't even exist in V6.