starred/planka
1
0
Fork 0
mirror of https://github.com/plankanban/planka.git synced 2026-02-24 19:08:59 +03:00
Code Issues 396 Packages Projects Releases 16 Wiki Activity

[PR #676] [MERGED] Slack bot integration #1063

New Issue
Closed
opened 2026-02-04 21:47:33 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-04 21:47:33 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/plankanban/planka/pull/676
Author: @mattboll
Created: 4/5/2024
Status: ✅ Merged
Merged: 4/8/2024
Merged by: @meltyshev

Base: master ← Head: slack-bot-integration

📝 Commits (10+)

  • 5d95527 added new custom service with slack integration helper functions; added axios package; added notifications for card create, delete, and update (move); added notifications for comment create
  • 7dba56c removed test webhook url
  • 35adf2a Replace plankaProdUrl with one set by environment. Remove hardcoded channel and unused variables.
  • 8310c41 Add slack variables to docker-compose
  • 3b2f47e Rename custom.js -> slack.js
  • ff8f861 updated to use currentUser.name for messages
  • 9b6766c Remove unused code and comments
  • 4e02efa chore: replace axios by fetch
  • b1ad354 refactor: extract code from controllers to helpers
  • cee4129 fix: Refactoring

📊 Changes

9 files changed (+129 additions, -24 deletions)

View changed files

📝 docker-compose.yml (+11 -8)
📝 server/.env.sample (+11 -8)
📝 server/api/controllers/cards/delete.js (+1 -0)
📝 server/api/helpers/actions/create-one.js (+28 -0)
📝 server/api/helpers/cards/delete-one.js (+12 -0)
📝 server/api/helpers/notifications/create-one.js (+2 -0)
📝 server/api/helpers/utils/send-email.js (+1 -1)
➕ server/api/helpers/utils/send-slack-message.js (+53 -0)
📝 server/config/custom.js (+10 -7)

📄 Description

Hi,
I've refactored the PR #573 so I hope it will be faster for you to merge it

I rebased from master, I replaced axios by fetch, moved work from controller to helpers

Having i18n here would need (a lot of?) extra work

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/plankanban/planka/pull/676 **Author:** [@mattboll](https://github.com/mattboll) **Created:** 4/5/2024 **Status:** ✅ Merged **Merged:** 4/8/2024 **Merged by:** [@meltyshev](https://github.com/meltyshev) **Base:** `master` ← **Head:** `slack-bot-integration` --- ### 📝 Commits (10+) - [`5d95527`](https://github.com/plankanban/planka/commit/5d955272c8474efd44482f0e33bf8e8f9835a90e) added new custom service with slack integration helper functions; added axios package; added notifications for card create, delete, and update (move); added notifications for comment create - [`7dba56c`](https://github.com/plankanban/planka/commit/7dba56c6e196876b530507d0c27384544936a952) removed test webhook url - [`35adf2a`](https://github.com/plankanban/planka/commit/35adf2a95ad6bd33c0e6f82975df4b4825125ac4) Replace plankaProdUrl with one set by environment. Remove hardcoded channel and unused variables. - [`8310c41`](https://github.com/plankanban/planka/commit/8310c419afa974e2bccd802348eb5016c161fc0b) Add slack variables to docker-compose - [`3b2f47e`](https://github.com/plankanban/planka/commit/3b2f47e106de9c0a62d7c425f2f207360661dce6) Rename custom.js -> slack.js - [`ff8f861`](https://github.com/plankanban/planka/commit/ff8f8614a5cfb4c843a560ab6fcdf98a2e75e97a) updated to use currentUser.name for messages - [`9b6766c`](https://github.com/plankanban/planka/commit/9b6766cff6d960f08ee35680c1de142b8d1b4c82) Remove unused code and comments - [`4e02efa`](https://github.com/plankanban/planka/commit/4e02efa0ee9c2aff0a1e5810db4eee1d5028187c) chore: replace axios by fetch - [`b1ad354`](https://github.com/plankanban/planka/commit/b1ad354815cf3b65f7f1a389e446ed5f320092cf) refactor: extract code from controllers to helpers - [`cee4129`](https://github.com/plankanban/planka/commit/cee4129c165afaa913e0dfd65e69557ecad91039) fix: Refactoring ### 📊 Changes **9 files changed** (+129 additions, -24 deletions) <details> <summary>View changed files</summary> 📝 `docker-compose.yml` (+11 -8) 📝 `server/.env.sample` (+11 -8) 📝 `server/api/controllers/cards/delete.js` (+1 -0) 📝 `server/api/helpers/actions/create-one.js` (+28 -0) 📝 `server/api/helpers/cards/delete-one.js` (+12 -0) 📝 `server/api/helpers/notifications/create-one.js` (+2 -0) 📝 `server/api/helpers/utils/send-email.js` (+1 -1) ➕ `server/api/helpers/utils/send-slack-message.js` (+53 -0) 📝 `server/config/custom.js` (+10 -7) </details> ### 📄 Description Hi, I've refactored the PR #573 so I hope it will be faster for you to merge it I rebased from master, I replaced axios by fetch, moved work from controller to helpers Having i18n here would need (a lot of?) extra work --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-04 21:47:33 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
bug
documentation
documentation
duplicate
enhancement
good first issue
help wanted
pull-request
Mirrored from GitHub Pull Request
 question
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/planka#1063
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?