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

Make it possible to customize (shorten) the message #23

Open
alenkacz opened this issue Mar 14, 2016 · 3 comments
Open

Make it possible to customize (shorten) the message #23

alenkacz opened this issue Mar 14, 2016 · 3 comments

Comments

@alenkacz
Copy link

For me, this is too verbose, I would prefer to be able to specify less verbose message (without the build, status, just the first line with a link).

If it makes sense to you, I can contribute.

Example of a Slack notification as it is done right now:

Project :: Deploy to Canary  #164 started
Build Status
Project
----------------
Project :: Deploy to Canary
Build
----------------
164
Status
----------------
started
@cmakara
Copy link

cmakara commented Mar 24, 2016

You can have an attachmentless version of notifications like this:

❌ Modules :: testmodule [feature/something] #4 failed: Exit code 255

To achieve this you have to set the teamcity.notification.slack.useAttachment internal property to false in TeamCity.
For more info see this link.

@eljobe
Copy link
Contributor

eljobe commented Sep 19, 2016

I'd like to submit a patch to this repo which adds another per-user property which allows the user to configure whether or not they include the "attachment" in their notifications. It could be a simple textual field which accepts "TRUE,FALSE,YES,NO,true,false,yes,no,True,False,Yes,No" and defaults to "True"

The display name could be "Verbose Messages" and it would have the effect of using a SlackWrapper constructed with "useAttachements" having the boolean version of whatever the user enters. Sure, the better thing would be to make a jsp snippet which uses a checkbox instead of a text field, but this is really easy to implement.

I'll put together a pull request if you're interested.

@aclark09
Copy link
Contributor

Sounds good to me, maybe one day we can get a checkbox there, but definitely not needed. It'll be nice to just get this feature out there for people to use.

@eljobe eljobe mentioned this issue Sep 19, 2016
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

No branches or pull requests

4 participants