Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Post direct message #77

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

VitalyKrivoshapkin
Copy link

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

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

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

@VitalyKrivoshapkin
Copy link
Author

@sciyoshi important fix)

@sciyoshi
Copy link
Owner

@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.

@VitalyKrivoshapkin
Copy link
Author

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

@jeacott
Copy link

jeacott commented Feb 12, 2016

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.

@bluepioupiou
Copy link

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

@VitalyKrivoshapkin
Copy link
Author

@bluepioupiou I'm using in our company succesfully)

@VitalyKrivoshapkin
Copy link
Author

@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

@eastuni
Copy link

eastuni commented Sep 11, 2017

merged #124

@lafraia
Copy link

lafraia commented Dec 29, 2017

@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.

cat-in-136 added a commit to cat-in-136/redmine-slack that referenced this pull request Aug 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants