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

feat: send slack notifications to authorized workspaces #37

Merged
merged 12 commits into from
Aug 5, 2021

Conversation

whoAbhishekSah
Copy link
Member

No description provided.

whoAbhishekSah and others added 9 commits July 29, 2021 12:54
Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
@whoAbhishekSah whoAbhishekSah linked an issue Jul 29, 2021 that may be closed by this pull request
4 tasks
@pyadav pyadav requested a review from AkarshSatija July 29, 2021 12:45
@ravisuhag ravisuhag changed the title send slack notifications to authorized workspaces using Siren API feat: send slack notifications to authorized workspaces Jul 29, 2021
Co-authored-by: Abhishek <abhi.sah.97@gmail.com>
api/handlers/notifier.go Outdated Show resolved Hide resolved
domain/notifier.go Outdated Show resolved Hide resolved
pyadav and others added 2 commits August 5, 2021 10:46
Co-authored-by: Abhishek <abhi.sah.97@gmail.com>
Co-authored-by: Abhishek <abhi.sah.97@gmail.com>
@pyadav pyadav force-pushed the feat/send_slack_message branch from 5a9dbfa to 4c343cd Compare August 5, 2021 07:17
@AkarshSatija AkarshSatija merged commit 1bc391b into main Aug 5, 2021
@whoAbhishekSah whoAbhishekSah deleted the feat/send_slack_message branch August 11, 2021 07:28
ravisuhag pushed a commit that referenced this pull request Aug 21, 2022
* feat: setup api to send slack notification

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* feat: send notifications to users and joined public/private channels

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: fix failing tests

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: refactor and test slack notifier client

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: refactor and test slack notifier service

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: slack notifier handler

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: fetching access token for a workspace

* docs: add swagger schema

* feat: add request payload validation in notifications api handler

* refactor: move validation into domain

Co-authored-by: Abhishek <abhi.sah.97@gmail.com>

* refactor: custom validation using oneof tag
Co-authored-by: Abhishek <abhi.sah.97@gmail.com>

* refactor: use custom error

Co-authored-by: Abhishek <abhi.sah.97@gmail.com>

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
ravisuhag pushed a commit that referenced this pull request Aug 21, 2022
* feat: setup api to send slack notification

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* feat: send notifications to users and joined public/private channels

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: fix failing tests

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: refactor and test slack notifier client

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: refactor and test slack notifier service

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: slack notifier handler

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>

* test: fetching access token for a workspace

* docs: add swagger schema

* feat: add request payload validation in notifications api handler

* refactor: move validation into domain

Co-authored-by: Abhishek <abhi.sah.97@gmail.com>

* refactor: custom validation using oneof tag
Co-authored-by: Abhishek <abhi.sah.97@gmail.com>

* refactor: use custom error

Co-authored-by: Abhishek <abhi.sah.97@gmail.com>

Co-authored-by: Praveen Yadav <pyadav9678@gmail.com>
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.

Support slack notification
3 participants