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

ability to send notification to different slack workspaces #39

Closed
bsushmith opened this issue May 22, 2023 · 0 comments · Fixed by #67
Closed

ability to send notification to different slack workspaces #39

bsushmith opened this issue May 22, 2023 · 0 comments · Fixed by #67
Assignees
Labels
enhancement New feature or request notifier: slack

Comments

@bsushmith
Copy link
Collaborator

Summary
Currently, slack notifier is configured at server level. So for one deployment of guardian, there can only be one slack workspace configured using the access token.
But if there are users who are on different slack workspaces (which can happen in a huge organization), sending notification to only one workspace wouldn't work.

For example -
If the guardian server is set up with slack server - workspaceA, and there is only deployment of guardian in the org which has multiple slack workspaces - workspaceA, workspaceB, workspaceC, workspaceD, etc.

If a user who is only in workspaceB, raises an appeal - the notification for the appeal and access get sent to workspaceA, but since the user is not in workspaceA - the notification will error out and the user wouldn't receive any notifications.

@bsushmith bsushmith added enhancement New feature or request notifier: slack labels Jun 8, 2023
lifosmin pushed a commit to lifosmin/guardian that referenced this issue Aug 31, 2023
* feat(siren): filter receivers by labels

* feat(siren): add receiver parent_id field
@bsushmith bsushmith linked a pull request Sep 23, 2023 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request notifier: slack
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants