redmine-slack icon indicating copy to clipboard operation
redmine-slack copied to clipboard

Post direct message

Open VitalyKrivoshapkin opened this issue 9 years ago • 10 comments

For success posting to direct user, login value in redmine must have same username from Slack. Changing redmine login value is acceptably.

VitalyKrivoshapkin avatar Jan 20 '16 08:01 VitalyKrivoshapkin

@sciyoshi In second commit was added a filter for direct posts: Send direct post if issue was modified not by assignee user

VitalyKrivoshapkin avatar Jan 22 '16 06:01 VitalyKrivoshapkin

@sciyoshi important fix)

VitalyKrivoshapkin avatar Jan 26 '16 01:01 VitalyKrivoshapkin

@VitalyKrivoshapkin thank you for the patch - this is a very useful feature! One question I have: is there ever a case where the assignee would want to be notified for changes that they make? I'm asking if the second option (direct_speak_rule) is necessary and if the behavior of Filter1 could simply be the default.

sciyoshi avatar Jan 26 '16 03:01 sciyoshi

@sciyoshi You're right. Let's do Filter1 to default. I'll commit it

VitalyKrivoshapkin avatar Jan 26 '16 04:01 VitalyKrivoshapkin

please resolve this - I want this feature a lot!! - a place to support redmine user->slack user mappings would be super helpful in the configuration too.

jeacott avatar Feb 12 '16 02:02 jeacott

+1 on this feature, can someone look into this ?

bluepioupiou avatar Mar 04 '16 08:03 bluepioupiou

@bluepioupiou I'm using in our company succesfully)

VitalyKrivoshapkin avatar Mar 04 '16 13:03 VitalyKrivoshapkin

@sciyoshi @jeacott @bluepioupiou pull was updated. Added: send direct message to old user that he was aware of when him task was moved to other user

VitalyKrivoshapkin avatar Jun 16 '16 07:06 VitalyKrivoshapkin

merged #124

eastuni avatar Sep 11 '17 07:09 eastuni

@eastuni This is a great feature (including notifications for watchers or mentions in updates). I would use the same kind of preferences as the ones used for email notifications.

Thanks! Look forward for this to be merged in master.

lafraia avatar Dec 29 '17 00:12 lafraia