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

Send message using attachment with color. #20

Merged
merged 1 commit into from
Sep 7, 2014

Conversation

hSATAC
Copy link
Contributor

@hSATAC hSATAC commented Aug 20, 2014

Fixes #13

screenshot:

2014-08-21 12 33 05

@cloudbees-pull-request-builder

plugins » slack-plugin #21 SUCCESS
This pull request looks good

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@tonybruess
Copy link

@grantmd Any chance of getting this reviewed and merged?

@grantmd
Copy link
Contributor

grantmd commented Sep 7, 2014

I have not tested this, but it looks clean. Thank you for the changes, will merge.

Assuming my maven install cooperates, I'll push a new version as well.

@grantmd grantmd closed this Sep 7, 2014
@grantmd grantmd reopened this Sep 7, 2014
grantmd added a commit that referenced this pull request Sep 7, 2014
Send message using attachment with color.
@grantmd grantmd merged commit 21b7ff1 into jenkinsci:master Sep 7, 2014
@grantmd
Copy link
Contributor

grantmd commented Sep 7, 2014

It did not behave. Will try again in the morning

@cloudbees-pull-request-builder

plugins » slack-plugin #22 SUCCESS
This pull request looks good

@hSATAC
Copy link
Contributor Author

hSATAC commented Sep 7, 2014

It's working on my build. If you could provide more information I'd love to help debugging it.

samrocketman added a commit that referenced this pull request Feb 10, 2015
New features:

* Advanced settings
  * Team domain and token can be specified per project; falls back to
    global config if not specified (#19)
  * Post a list of commits with the build notification (title and author
    of commit) (#30, #44, #45)
  * Include JUnit test summary in build notifications (#25)
* Use colors when sending a message using slack (#20, #24).  Also, the
  start notifications use the color of the last build result (#43)
* Support for authenticated proxies (#14)
* Test Connection button (#46, #28)
* Option to disable subsequent notifications on build failures (#46, #15)

Improvements:

* Report 'return to normal' on transition from unstable to success (#21)
* Improved logging.

Bug fixes:

* When changing the global slack settings the slack config in jobs are
  updated as well (#26, #12)
* Fix NullPointerException and output log message if slack is not
  configured (#35, JENKINS-26066)
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.

Set status color in Slack message border
5 participants