search

matrix-org / matrix-appservice-discord

A bridge between Matrix and Discord.
Apache License 2.0
804 stars 152 forks source link

feat: add Matrix -> Discord reactions #877

Open mangofeet opened 1 year ago

mangofeet commented 1 year ago

Implements reactions from matrix to discord, intended for cases where a single matrix user is using the bot as a personal puppet

Functionality is disabled by default and enable by setting bridge.enableMatrixReactions to true

mangofeet commented 1 year ago

Looking forward to Discord reactions. clap Do you have everything you need to finish the PR?

I still need to figure out how to respond to the redaction of a reaction in Matrix. Unfortunately it looks like I may have to figure out a way to store the records differently so that the event ID of the reaction that comes back in event.redacts can be used to look up the original Matrix event or at least the original Discord message, and it needs to save the emoji used so that it can remove the correct one I believe.

I'll be checking out the code that's in #862 to maybe pull some implementation details from there (hopefully without depdending on that PR since it's still in review I guess)

mangofeet commented 1 year ago

OK, so I got reactions redacting from matrix to discord. Still need to figure out the edge case of the thumbs up, but otherwise it's working as I intended now.

mangofeet commented 1 year ago

Handled the thumbs up exception from Matrix to Discord, but not the other way around yet, not sure if that's strictly necessary to merge this though. It may be better to get this and #862 merged, and then handle any emoji exceptions in a unified manner after the fact.