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

Add slack channel notification integration #573

Merged
merged 8 commits into from Apr 7, 2024

Conversation

GlitchWitch
Copy link
Contributor

@GlitchWitch GlitchWitch commented Dec 23, 2023

Contributions primarily made by @chembrad44 of GlitchSecure


This PR adds a simple notification integration which will alert a specified slack channel on various actions.

image

A user must create a Slack App in their Slack instance with the chat:write scope and provide both the SLACK_BOT_TOKEN for the app and the SLACK_CHANNEL_ID for the channel they wish to have notifications arrive in.

Creating a slack app

  1. Visit https://api.slack.com/apps
  2. Choose "Create New App"->"From scratch"
  3. Select your workspace
  4. Visit the "OAuth & Permissions" tab
  5. Select "Add an OAuth Scope" and enter chat:write
  6. Install the app to your workspace and invite the bot to your channel
  7. Copy the "Bot User OAuth Token" to the SLACK_BOT_TOKEN environment variable and the Channel ID to the SLACK_CHANNEL_ID environment variable.

@CLAassistant
Copy link

CLAassistant commented Dec 23, 2023

CLA assistant check
All committers have signed the CLA.

@meltyshev
Copy link
Member

Hi! Thank you for your contribution 🙏
We'll be sure to take a look as soon as we get back on track after the holidays. Happy New Year 🎄

@jalibu
Copy link

jalibu commented Jan 12, 2024

Pleeeeeease use that opportunity to add outbound notifications (also email) as a general feature into Planka 🙏

That's just the missing piece for perfection!

@gayathridevii
Copy link

Any updates on this?

@meltyshev
Copy link
Member

I apologize for taking so long, trying to make time for this. I can't accept right away as I need to make some changes. For example, move notifications sending from controllers to helpers (just to keep the same code structure), use fetch instead of additional axios. We've also thought about being able to send notifications based on the language selected by a user, but it can be added later.

@mattboll mattboll mentioned this pull request Apr 5, 2024
@meltyshev
Copy link
Member

Merged: #676

@meltyshev meltyshev closed this Apr 7, 2024
@meltyshev
Copy link
Member

meltyshev commented Apr 7, 2024

Damn, I really apologize, due to squash merge and other PR with refactoring you weren't listed as contributors :( Trying to figure out how this can be fixed...

UPD: fixed 🎉

@meltyshev meltyshev reopened this Apr 7, 2024
@meltyshev meltyshev merged commit 273d994 into plankanban:master Apr 7, 2024
1 check passed
@GlitchWitch GlitchWitch deleted the slack-bot-integration branch April 23, 2024 17:11
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.

None yet

6 participants