matrix-org / matrix-appservice-slack

A Matrix <--> Slack bridge
Apache License 2.0
276 stars 72 forks source link

Handle deactivated and reactivated Slack installations #480

Open Half-Shot opened 4 years ago

Half-Shot commented 4 years ago

If the bridge Slack app becomes deactivated on a team, we should make the room links dormant. If and when the app becomes active again, we should try to activate the rooms.

As it currently stands when the app is deactivated and reactivated, the team RTM connection is not and the BridgedRoom objects are not refreshed so the botClient attached will not work.

juliusv commented 3 years ago

Is this the reason why when I remove bridging (including the Element app on the Slack side) and then re-add it, messages from Slack->Matrix no longer work? Everything worked when I set bridging up the first time, but then for testing I tore everything down and re-added it, but now communication only works one way (the Element Bridge is invited to the correct Slack channel).

vbatts commented 3 years ago

i believe this is affecting us as well. It's currently like the matrix bridge is read-only (nothing from slack is sent to matrix).