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

Notifications: Mattermost #2231

Open
robinfrick opened this issue Mar 24, 2016 · 23 comments
Open

Notifications: Mattermost #2231

robinfrick opened this issue Mar 24, 2016 · 23 comments
Labels
Priority:P2 Average run of the mill bug Reporting/Pulses Now called Subscriptions Type:New Feature

Comments

@robinfrick
Copy link

robinfrick commented Mar 24, 2016

It'd be nice if besides slack we could also have pulses in Mattermost.

Related to #2298

⬇️ Please click the 👍 reaction instead of leaving a +1 or update? comment

@mazameli mazameli changed the title Feature Request: Add Mattermost beside Slack Allow Pulses in Mattermost Mar 24, 2016
@salsakran
Copy link
Contributor

@robinfrick Do you have any experience with the Mattermost API? Is it a mostly 1-1 mapping of the Slack bot/slash command/app api structure? Would you be willing to do a bit of research to help us understand how much overlap there would be in the current way we post images to Slack with how we could post to Mattermost?

@tlrobinson
Copy link
Contributor

FYI the Slack endpoints we currently use are:

  • chat.postMessage - actually posts pulses with attached images
  • users.list - list users for destination selection widget
  • channels.list - list channels for destination selection widget
  • channels.create - workaround for posting files, creates a hidden channel where we upload images
  • rtm.start - start a streaming connection for MetaBot

@tlrobinson
Copy link
Contributor

And yeah I think it would be super neat if we created a simple abstraction so it's easy for various messaging services to plug into Pulses + MetaBot, but we don't want to be stuck with the lowest common denominator either. Right now all we need is:

  1. list users and channels where you might want to post your pulse to
  2. post messages with attached images to any of those users or channels
  3. for MetaBot, listen to a stream of messages on any/every channel and respond with text or pulses

But we might also want to do more in the future.

@robinfrick
Copy link
Author

Ok as far as I can see it there is not an option for 1. The person setting up the pulse would have to type the channel name as Metabase wouldn't be able to get a list of channel/user names from mattermost. 2. & 3. look like they are supported.

For metabot:
The way integrations work on Mattermost is that you allow an integration to listen for keywords in a specific channel not everywhere at once. So that's another limitation.
http://docs.mattermost.com/developer/webhooks-outgoing.html

For pulses:
This should be better. The only limitation I see is that users have to type or copy-paste the channel names and there is no way of knowing whether it is correct without testing the pulse, so there is no immediate feedback.
http://docs.mattermost.com/developer/webhooks-incoming.html

When I wrote the request I wasn't aware of Metabot yet. I think Pulses should be relatively straight forward to adapt to Mattermosts API. I'm far from being a programmer, but I got a little bot that can post messages to mattermost up and running within very little time. I have never tried the same for slack so I can't judge, but according to mattermost's documentation at least the message format is the same. But as far as I can see it no one else +1'd my request yet so a) I can understand if you say it's not worth the effort if barely anyone wants it and b) having an incomplete experience compared to slack with metabot would probably rather raise more requests from users and possibly lead to disappointment rather than happy users.

@camsaul camsaul added the Reporting/Pulses Now called Subscriptions label Apr 4, 2016
@davidsirr

This comment has been minimized.

@camsaul camsaul changed the title Allow Pulses in Mattermost Support Mattermost (Alternative to Slack) Integration Apr 13, 2016
@camsaul
Copy link
Member

camsaul commented Jun 12, 2017

Hey everyone, one way we prioritize issues is by sorting them by the number of 👍 reactions on each issue's description. So instead of leaving +1 or 👍 comments, please just upvote by adding a reaction to the issue description.

@fernandosjp

This comment has been minimized.

@logros
Copy link

logros commented Jan 5, 2018

Hi,
i use zabbix - mattermost integration via script https://github.com/muokata/zabbix-mattermost-alertscript/blob/master/mattermost.sh,

All it's ok, webhook work almost full, log from console:
Входящий вебхук получен. Содержание={"icon_url": "http://a.slack-edge.com/41b0a/img/plugins/_skeleton/service_36.png", "attachments": [ {"color": "#00ff33", "text": "OK: Много выпадающих пакетов host_pinger_ул.Александра Довженка(ул.Урицкого),5,dlink dir-100"} ], "channel": "#logros", "username": "ZABBIX Informer", "icon_emoji": ":ok_hand:"}

Messages come to chat, but icon and emoji not shown.
what's wrong with my config, please help.

@mauricio-sonegatti

This comment has been minimized.

@hawaiik

This comment has been minimized.

@gsimonneaux-oceanet

This comment has been minimized.

@sidneiweber

This comment has been minimized.

@mazameli
Copy link
Contributor

Folks, we'd appreciate it if you could please utilize the thumbs-up reaction in the original issue description to show your support for this or any other issue, rather than +1 for x comments. That gives us a sortable way to see what matters most to the community, and keeps issue spam to a minimum. Cheers.

@mazameli
Copy link
Contributor

This issue is for Mattermost integration. If you'd like to voice support for Rocket Chat integration, please upvote this issue: #10719

Otherwise, we're going to think that you're voicing support for Mattermost.

But "upvote," we mean please leave a 👍 reaction on the description of the issue. Please do not leave comments simply saying "+1", as that will cause everyone watching the issue to get a needless notification. Thanks!

@salsakran salsakran added this to The Future in Metabase Project Roadmap Mar 5, 2020
@salsakran salsakran moved this from The Future to Bigger Projects in Metabase Project Roadmap Mar 5, 2020
@salsakran salsakran moved this from Bigger Projects to Backlog in Metabase Project Roadmap Mar 10, 2020
@mazameli mazameli added Priority:P2 Average run of the mill bug and removed .Proposal/Being Discussed labels Apr 29, 2020
@narendrapsgim

This comment has been minimized.

@ab-smith

This comment has been minimized.

1 similar comment
@vapolonio

This comment has been minimized.

@mahenzon

This comment has been minimized.

@adjenks
Copy link

adjenks commented Sep 22, 2021

@mahenzon @ab-smith @vapolonio @narendrapsgim

Please click the 👍 reaction instead of leaving a +1 comment

@petitalb

This comment was marked as duplicate.

@brunobergher brunobergher changed the title Support Mattermost (Alternative to Slack) Integration Notifications: Mattermost Integration Jan 11, 2023
@brunobergher brunobergher changed the title Notifications: Mattermost Integration Notifications: Mattermost Jan 13, 2023
@nosovk
Copy link

nosovk commented May 31, 2023

I want to admit that Mattermost mimics to slack API. This means that current slack integration should just work, if you will allow changing a URL in it.

@paoliniluis
Copy link
Contributor

@nosovk are you completely sure about this?

@nosovk
Copy link

nosovk commented Jun 1, 2023

yep, checkout the link: https://jeffschering.github.io/mmdocs/monolith/developer/api.html#slack-compatible-webhooks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:P2 Average run of the mill bug Reporting/Pulses Now called Subscriptions Type:New Feature
Projects
None yet
Development

No branches or pull requests