matrix-appservice-discord icon indicating copy to clipboard operation
matrix-appservice-discord copied to clipboard

Discord reply support

Open StepS- opened this issue 5 years ago • 6 comments

The bridge will need to account for Discord users replying to other messages. (no source message/sender will currently be shown)

StepS- avatar Nov 20 '20 21:11 StepS-

discord.js will first need to gain inline reply support of course - it doesn't look like they even have a tracking issue for it yet.

auscompgeek avatar Nov 20 '20 23:11 auscompgeek

Looks like there's a PR though: discordjs/discord.js#4874

auscompgeek avatar Nov 21 '20 00:11 auscompgeek

So it looks like it's possible currently (without updating the library) to determine whether a Discord message is a reply by looking at Message#reference. So we can translate Discord replies to Matrix replies.

It would be good to send Discord replies at some point though. But that's blocked on discord.js bikeshedding.

auscompgeek avatar Nov 29 '20 09:11 auscompgeek

The discord.js PR finally got merged :tada:

Would it be worth merging the upstream changes into better-discord.js and making a prerelease for the bridge?

auscompgeek avatar Dec 09 '20 02:12 auscompgeek

Any updates whether the bridge is going to support Discord replies?

HansJK avatar Feb 23 '21 17:02 HansJK

  • [ ] Discord->Matrix: #706 (Fixed in #704)
  • [ ] Matrix->Discord: #693

While Matrix->Discord replies are a bit trickier, as webhooks do not support replies, one can use buttons instead. See Connections Bot's implementation: image

rollingmoai avatar Sep 03 '22 04:09 rollingmoai