Slack bridge: Update Slack bridge with Events API.

PieterCK commented 3 weeks ago

Currently we use the Slacks legacy RTM API as the "listener" from Slack to Zulip. This commit updates our Slack Bridge to use Webhook integration to get messages across from Slack to Zulip.

We use our Slack webhook integration for that part of the bridge, it is being updated to use the latest Event API at PR#30465. So, instead the changes in this PR are only the following:

Updating the documentation on how to setup the updated Slack Bridge
Adding backwards compatibility by adding the "--legacy" parameter to turn on the RTM API
Make the bridge compatible with the Slack Webhook integration by preventing looping messages.

Fixes #825.

PieterCK commented 1 week ago

@zulipbot add "buddy review"

@kenneth please review this please, thank you

zulipbot commented 1 week ago

ERROR: Label "buddy review" does not exist and was thus not added to this pull request.

PieterCK commented 5 days ago

Update: Addressed review and dropped support for RTM API

PieterCK commented 2 days ago

@sbansal1999 I think this one's also ready for mentor review, please do check it out! Thanks

zulip / python-zulip-api

Slack bridge: Update Slack bridge with Events API. #826