Skip to content

Slack notif facelift - #577

Merged
RoiGlinik merged 5 commits into
masterfrom
slack-notif-facelift
Oct 20, 2022
Merged

Slack notif facelift#577
RoiGlinik merged 5 commits into
masterfrom
slack-notif-facelift

Conversation

@RoiGlinik

Copy link
Copy Markdown
Contributor

Also some style changes from the formatter

@RoiGlinik
RoiGlinik requested a review from Sheeproid October 19, 2022 07:50
@RoiGlinik

Copy link
Copy Markdown
Contributor Author

image

image

@RoiGlinik

Copy link
Copy Markdown
Contributor Author

image

It is now easier to skim over alerts and get the gist of them.

@Sheeproid Sheeproid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread src/robusta/core/reporting/base.py
Comment thread src/robusta/integrations/slack/sender.py Outdated
Comment thread src/robusta/integrations/slack/sender.py Outdated
@RoiGlinik
RoiGlinik merged commit 719580b into master Oct 20, 2022
@RoiGlinik
RoiGlinik deleted the slack-notif-facelift branch October 20, 2022 07:39
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.

2 participants