Skip to content

Reactions setting#6285

Closed
kable-wilmoth wants to merge 4 commits intoRocketChat:developfrom
kable-wilmoth:ReactionSetting
Closed

Reactions setting#6285
kable-wilmoth wants to merge 4 commits intoRocketChat:developfrom
kable-wilmoth:ReactionSetting

Conversation

@kable-wilmoth
Copy link

@kable-wilmoth kable-wilmoth commented Mar 8, 2017

@RocketChat/core

Closes #6044
Closes RocketChat/feature-requests#620 (possibly)

Sorry, this PR is built on top of #6282, so if that one is accepted, then please focus on commit 159d19c

Added a setting to enable/disable reactions between slack and rocket

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-ordered the SlackBridge settings. Put the most probably used settings toward the top

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just re-used this i18nLabel

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added these here at the top for quick access to properties being used in SlackBridge

@engelgabriel engelgabriel added this to the 0.54.0 milestone Mar 8, 2017
Introduced ES2015 Import/Export to resolve dependencies
@engelgabriel engelgabriel modified the milestones: 0.54.0, 0.55.0 Mar 22, 2017
@engelgabriel engelgabriel modified the milestones: 0.55.0, 0.56.0 Apr 13, 2017
@engelgabriel engelgabriel modified the milestones: 0.56.0, 0.57.0 May 9, 2017
@engelgabriel engelgabriel modified the milestones: 0.57.0, 0.58.0 Jun 19, 2017
@rodrigok rodrigok modified the milestones: 0.58.0, 0.59.0 Aug 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Slack integration: Reactions made by the bot user Slackbridge reactions - disable 2 way sync. Fet Req

5 participants

Comments