Skip to content

Conversation

@cody-eding
Copy link
Contributor

@cody-eding cody-eding commented Oct 28, 2025

close #5421

📑 Description

This pull request adds Zoom Chat as a supported provider.

The provider supports sending a message to a configured Zoom Chat channel using the Zoom Incoming Webhook app. Optionally, the provider can be configured with Zoom application credentials to allow for tagging Zoom users in the notification.

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Oct 28, 2025

Skipped: This PR does not target one of your configured branches: (refactor/2512-nextjs-15)

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Oct 28, 2025
@vercel
Copy link

vercel bot commented Oct 28, 2025

@cody-eding is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Oct 28, 2025

CLA assistant check
All committers have signed the CLA.

@dosubot dosubot bot added Documentation Improvements or additions to documentation Feature A new feature Provider Providers related issues labels Oct 28, 2025
@cody-eding cody-eding force-pushed the feature/zoomchatprovider branch from b6bb3e2 to 1f32627 Compare October 28, 2025 15:01
@shahargl
Copy link
Member

hey @cody-eding !

thanks for this contribution.

  1. can you create an issue and assign the issue to this PR? (close #issue)
  2. can you add some short video demonstrating this provider?
    thanks!

@cody-eding
Copy link
Contributor Author

close #5421

@cody-eding cody-eding force-pushed the feature/zoomchatprovider branch 3 times, most recently from 1f95769 to 3175e67 Compare October 28, 2025 18:42
@cody-eding cody-eding force-pushed the feature/zoomchatprovider branch from 3175e67 to 6110fd6 Compare October 28, 2025 18:46
@cody-eding
Copy link
Contributor Author

Thanks, @shahargl.

I corrected a few other small items from my initial PR as well.

A video of the basic functionality is at: https://youtu.be/p9FPb-Cu8-s

If you setup the provider with a Zoom application ID capable of getting Zoom user information, you can also tag individuals in the messages (equivalent to using @ in a Zoom message). This can be useful if you have all alerts for a channel muted except for direct mentions.

@shahargl
Copy link
Member

Hey! thanks for opening this PR!

We have some problem with the CI/CD. Going to handle it soon and let this PR in :)

@shahargl shahargl enabled auto-merge (squash) November 16, 2025 11:38
@vercel
Copy link

vercel bot commented Nov 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
keep Ready Ready Preview Comment Nov 16, 2025 11:44am

@shahargl shahargl self-requested a review November 16, 2025 11:38
Copy link
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

lgtm

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 16, 2025
@shahargl shahargl merged commit 1c37165 into keephq:main Nov 16, 2025
7 checks passed
@github-actions
Copy link
Contributor

💪 Fantastic work @cody-eding! Your very first PR to keep has been merged! 🎉🥳

You've just taken your first step into open-source, and we couldn't be happier to have you onboard. 🙌
If you're feeling adventurous, why not dive into another issue and keep contributing? The community would love to see more from you! 🚀

For any support, feel free to reach out on the community: https://slack.keephq.dev. Happy coding! 👩‍💻👨‍💻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Improvements or additions to documentation Feature A new feature lgtm This PR has been approved by a maintainer Provider Providers related issues size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🔌 Provider]: Zoom Chat Provider

3 participants