Rocket.Chat icon indicating copy to clipboard operation
Rocket.Chat copied to clipboard

[NEW] Improved SlackBridge features

Open nonth opened this issue 8 years ago • 5 comments

@RocketChat/core

This pull contains a bunch of improvements on SlackBridge package that listed below

Features

  • Add/Remove star message or file between RocketChat and Slack

  • Upload file from RocketChat to Slack or fallback to a msg link to file on RocketChat if user doesn't provide Slack's access_token

  • Added Slack setting for user to connect their Slack account to RocketChat in order to use Add/Remove star between RC and Slack or upload file from RC to Slack screen shot 2017-05-10 at 10 09 48 pm

  • Added more Slack settings for admin such as OAuth settings, Slack Event API screen shot 2017-05-10 at 10 10 57 pm

  • Improved convert between Slack msg text <-> RocketChat msg text

  • Improved display reply, quote from RocketChat to Slack using Slack's message attachment

Bug Fixed

  • Fixed cannot download image from Slack (#5791) we use Meteor.runAsUser to fixed this issue from meteor-collection-hooks's README shown that you cannot get userId in hook if you're executing from API

Noted

In order to use those features above we using Slack RTM and Event API you need to create Slack App to get information below

  1. Client ID
  2. Client Secret
  3. Redirect URL
  4. API Token (bot)
  5. Verification Token for Event API

and required user to connect their Slack's account with that app (My Account > Slack > Add to Slack)

because to be able to upload file or get Added/Removed star events from Slack it's required user access_token and permissions to access their data

nonth avatar May 10 '17 15:05 nonth

@nonth Can you fix the conflicts? @marceloschmidt or @graywolf336 can you review this PR?

rodrigok avatar May 17 '17 22:05 rodrigok

@rodrigok Done :)

nonth avatar May 18 '17 00:05 nonth

Hello, any progress on this?

We have used this patch for month without problem. Would love to have it merged (so we don't have to maintain separate fork of Rocket.chat).

somsak avatar Jun 09 '17 07:06 somsak

This patch seems to be the missing ingredient to allow easier slack-rocket integration... is there a chance this is going to be merged soon ?

rmdes avatar Dec 12 '17 16:12 rmdes

Related to https://github.com/RocketChat/Rocket.Chat/pull/8233

rodrigok avatar May 28 '20 19:05 rodrigok