search

alexkvak / teamcity-slack

TeamCity Slack plugin
MIT License
124 stars 23 forks source link

No preview available #36

Closed sandweel closed 5 years ago

sandweel commented 6 years ago

Hello. All notification has no preview - [no preview available]. I using default text from your example. Is it fixable by my side?

alexkvak commented 6 years ago

Hello, it's Slack application behaviour, while the message has no content, but only attachments.

sandweel commented 6 years ago

Yes I know but how I can move attachments to the message body or some text like a build status?

alexkvak commented 6 years ago

You cannot do this. It breaks the current view with vertical line, so I use attachments.

sandweel commented 6 years ago

I think if you'll add new block above the box Message template for text body it will be very helpful. I don't needed to open slack application that I could read build status.

sandweel commented 6 years ago

@alexkvak Could you please tell will you do any updates for showing preview build? Thank you in advance.

alexkvak commented 6 years ago

I've discussed it with my team mates and we don't see the right solution here. I've wrote to Slack twitter and the answer was: we don't plan to provide any api to specify exlicitly the message preview. So, I stay this issue open and wait for some other ideas.

j-j-marvin commented 5 years ago

I'd say that configuration whether to send message as attachment or plain text might solve the issue in some way. There will be at a least a choice between neat vertical view and preview of the message. @alexkvak does it make sense from your point of view?

alexkvak commented 5 years ago

This is interesting, we can use emoji ⛔️, ⚪️ and ✅ to indicate the build status. But I doubt that backward compatibility e.g. switching between message representation is needed. @adbrvn what’s your thoughts?

j-j-marvin commented 5 years ago

By the way I've checked the integration I have between WP and slack. It has both text (with emojis) and attachment with vertical. View. I believe that you can easily reach that if message.message is not empty here (SlackGatewayL130):

Try(x.sendMessage(channel, message.message, message.attachment.orNull, channelChatConfiguration))