Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

add MattermostWebhook notification block #8341

Merged
merged 5 commits into from
Feb 13, 2023
Merged

add MattermostWebhook notification block #8341

merged 5 commits into from
Feb 13, 2023

Conversation

zzstoatzz
Copy link
Collaborator

@zzstoatzz zzstoatzz commented Feb 1, 2023 •
edited
Loading

image

Adds a MatterMostWebhook notification block using the AbstractAppriseNotificationBlock, as requested in community.

Closes #8339

Example

In [1]: from prefect.blocks.notifications import MatterMostWebhook

In [2]: m = MatterMostWebhook(hostname="localhost", token="aksdfhjaskjfhsakdfjhkasd")

In [3]: m.notify("suuuuhhhh")

image

Checklist

  • This pull request references any related issue by including "closes <link to issue>"
    • If no issue exists and your change is not a small fix, please create an issue first.
  • This pull request includes tests or only affects documentation.
  • This pull request includes a label categorizing the change e.g. fix, feature, enhancement

@zzstoatzz zzstoatzz requested a review from a team as a code owner February 1, 2023 06:04
@zzstoatzz zzstoatzz self-assigned this Feb 1, 2023
@netlify
Copy link

netlify bot commented Feb 1, 2023 •
edited
Loading

✅ Deploy Preview for prefect-docs ready!

Name Link
🔨 Latest commit ca0fcbf
🔍 Latest deploy log https://app.netlify.com/sites/prefect-docs/deploys/63ea94a6cea08400087cec38
😎 Deploy Preview https://deploy-preview-8341--prefect-docs.netlify.app/api-ref/prefect/blocks/notifications
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@zzstoatzz zzstoatzz added feature A new feature integrations Related to integrations with other services labels Feb 1, 2023
@zzstoatzz zzstoatzz changed the title add MatterMostWebhook notification block add MattermostWebhook notification block Feb 1, 2023
@zzstoatzz zzstoatzz requested a review from desertaxle February 1, 2023 14:33
desertaxle
desertaxle requested changes Feb 1, 2023
View reviewed changes
@zzstoatzz zzstoatzz requested a review from desertaxle February 1, 2023 16:50
desertaxle
desertaxle approved these changes Feb 3, 2023
View reviewed changes
Copy link
Member

@desertaxle desertaxle left a comment

Choose a reason for hiding this comment

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

LGTM!

@zzstoatzz zzstoatzz merged commit f1cf110 into main Feb 13, 2023
@zzstoatzz zzstoatzz deleted the mattermost-notification-block branch February 13, 2023 20:27
@desertaxle desertaxle mentioned this pull request May 3, 2023
Open
3 tasks
@desertaxle desertaxle mentioned this pull request Jun 30, 2023
Merged
4 tasks
@kripnerl kripnerl mentioned this pull request Dec 11, 2024
Open
4 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@desertaxle desertaxle desertaxle approved these changes

@ahuang11 ahuang11 ahuang11 approved these changes

Assignees

@zzstoatzz zzstoatzz

Labels
feature A new feature integrations Related to integrations with other services
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add a MatterMost notification block
3 participants
@zzstoatzz @desertaxle @ahuang11