search

matrix-hacks / matrix-puppet-slack

puppet style slack bridge for matrix
47 stars 17 forks source link

The bridge ignores Slack emoji reactions of messages #60

Open thomas-profitt opened 5 years ago

thomas-profitt commented 5 years ago

Matrix doesn't support exactly this, but we could convert an emoji reaction to a Matrix message Replying to the original message with a single emoji and have the same semantics of an emoji reaction, for the Matrix client to do with as it pleases.

Even if we can't bridge this perfectly, we shouldn't miss out on the messages.

thomas-profitt commented 5 years ago

A screenshot showing how this looks in Matrix: screenshot_2018-11-17 riot

twouters commented 5 years ago

Depends on #58 ?

thomas-profitt commented 5 years ago

Partially, probably.

Slack doesn't send emoji reactions as Slack Thread messages, but we do need the ability to create Matrix Replies to implement this.

twouters commented 5 years ago

What I meant is that we need to implement a link between slack event timestamps and matrix event id's for both issues, once that's taken care of they will be easy to fix.

thomas-profitt commented 5 years ago

Oof, you're right about that.

michalrus commented 5 years ago

Matrix started supporting proper message reactions. :)