feat: use new Slack module to send slack notifications #1325

adityathebe · 2024-08-28T04:28:12Z

resolves: #923

needed quite a bit of refactoring because the templating of the notification title & body needs to be pushed to the very end of the pipeline instead of the very beginning. This is because now we need to template the message only after we've figured out the shoutrrr service we're sending to.

[skip ci]

notification/send.go

notification/templates/check.failed

adityathebe · 2024-08-28T12:26:39Z

* show labels * change outgoing_channel to channel

feat: use new Slack module to send slack notifications

434ff71

[skip ci]

adityathebe marked this pull request as draft August 28, 2024 04:28

adityathebe added 2 commits August 28, 2024 11:19

feat: implement slack blocks support

ae1a85b

[skip ci]

feat: add slack template funcs

7a7d680

[skip ci]

moshloop reviewed Aug 28, 2024

View reviewed changes

notification/send.go Outdated Show resolved Hide resolved

moshloop reviewed Aug 28, 2024

View reviewed changes

notification/templates/check.failed Show resolved Hide resolved

adityathebe marked this pull request as ready for review August 28, 2024 12:35

adityathebe requested a review from moshloop August 28, 2024 12:36

chore: update templates

c788742

* show labels * change outgoing_channel to channel

adityathebe force-pushed the feat/slack-notification-enhancements branch from 325ad46 to c788742 Compare August 28, 2024 13:46

moshloop merged commit 9b5a858 into main Aug 28, 2024
7 checks passed

moshloop deleted the feat/slack-notification-enhancements branch August 28, 2024 19:20

Provide feedback