-
Notifications
You must be signed in to change notification settings - Fork 79
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
Mention does not work #46
Comments
Hi, one of the issues we are seeing is that, on GitHub PRs, mentions seem to occur for anyone who has ever committed to the codebase at all. |
But i'm talking about other issue. Even if mention is correctly inserted into message there is no notification because mention is inside attachment (part of the message with red line to the left). |
@PeteGoo Should it have to work now? With latest release i don't receive mentions too |
I'll have a look |
@PeteGoo any updates? |
Can someone tell me if this is working now? Because there was a PR that should have fixed it. |
My bad :). I saw the issue was still Open and didn't read the full thread or see the PR. I see #68 was included in release 1.4.5; we're still running 1.4.4. Sounds like an update will fix this for us (unless there's something still not working, as per @galkinrost). I'll ping this thread if it doesn't work for me after the update. |
Thanks! Would be much appreciated! |
This appears to still be broken for us. TeamCity 9.1.5, tcSlackBuildNotifier 1.4.6. |
Thanks for the feedback! Can you show me what's wrong exactly? |
This is what I see (test names redacted): The user who broke the build (vgod) says he was not notified. I also find it interesting that the @mention text is black and non-clickable, whereas normal @mentions are blue and show the user's profile when clicked. I asked that user to confirm at I haven't yet tried to repro on my account, but if you'd like me to be 100% sure it's broken, I can try. I was poking around Slack API docs, and I found https://api.slack.com/docs/message-formatting#linking_to_channels_and_users which seems to suggest that you may need the user ID (beginning with To confirm this, I used the Slack API tester (https://api.slack.com/methods/chat.postMessage/test) to post two messages to a channel. The first had text The first created black text that did not notify me; the second created blue text that did notify me. Interestingly though, I just looked at what #68 changed; it supplies Browsing through the code a bit more, I see that (And even if adding |
Any update on this? |
I believe pull request #149 fixed this issue |
Yeah that looks like it'd fix the issue but there doesn't seem to be a recent release that includes that change. |
@IanYoder @laurencee sorry for the delay. Release is here: https://github.com/PeteGoo/tcSlackBuildNotifier/releases/tag/v1.4.7 I didn't retest this issue so if you could provide some feedback I'd close this issue afterwards. |
@marvin-w Unfortunately mentions still don't appear to be working, it's just basic |
@marvin-w @laurencee If I'm reading everything correctly, I don't think the change made it into the release. You can see here that line 333 of tcslackbuildnotifier-core/src/main/java/slacknotifications/SlackNotificationImpl.java appears to contain the fix, but that line wasn't changed as part of the release. I also wanted to say that I really appreciate your quick feedback and work! |
Oh... well that makes a lot of sense then! haha Edit: well unfortunately like i said, even with those changes it's not working for us :( |
@IanYoder @laurencee Thanks for your feedback. As far as I know the code from @waelouf was merged and is available in the current release: https://github.com/PeteGoo/tcSlackBuildNotifier/commits/v1.4.7 If it doesn't work there might be another issue which isn't fixed already. |
@marvin-w I see the change now - thank you. I'll let you know if it's working as soon as we're able to update the version and test it out. |
@marvin-w We were able to update the plugin today, and I confirmed that the notification worked - thank you! One suggestion I wanted to make, though: would it be possible to add some configuration to notify someone of subsequent failures, rather than just the initial failure? I didn't realize that the update was working right away, because I wasn't notified when I re-ran a broken build, or even when I committed another change; I had to fix the broken build, then re-break it. |
Mention of user that has broken the build does not work because it is written inside message attachment.
http://screencast.com/t/5eITBpGlJ
For mention to work it should be written in the text message before attachment.
The text was updated successfully, but these errors were encountered: