FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Implement slack integration service. by marcoblos · Pull Request #135 · python-gitlab/python-gitlab · GitHub

Repository navigation

Implement slack integration service. - #135

Closed
marcoblos wants to merge 5 commits into
python-gitlab:masterfrom
marcoblos:master
Closed

marcoblos wants to merge 5 commits into
python-gitlab:masterfrom
marcoblos:master

Conversation

Copy link
Copy Markdown

Implement method to put a new integration with Slack.

project.slack_service('https://hooks.slack.com/services/...', 'optional-username', '#optional-channel')

If not inform username and channel assumes the configured the slack service.

Copy link
Copy Markdown
Contributor

Thanks for this patch, but I'm afraid it breaks the CRUD logic that is used with other objects. I'd prefer to see a new ProjectSlackService class (or similar, that's just an idea) to manage the other methods (create/update/get/delete).

gpocentek closed this Aug 20, 2016

Copy link
Copy Markdown
Contributor

Support for services has been added. Thanks.

Copy link
Copy Markdown
Author

@gpocentek Thank's for add this feature and sorry for don't response you, i am very busy in last days.
I see your code and it is very nice! Awesome!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL