Skip to content

feat: add Trigv notification provider - #8178

Open
hchouhan wants to merge 1 commit into
louislam:masterfrom
hchouhan:feat/trigv-notification-provider
Open

hchouhan wants to merge 1 commit into
louislam:masterfrom
hchouhan:feat/trigv-notification-provider

Conversation

@hchouhan

Copy link
Copy Markdown

Summary

Adds Trigv (https://trigv.com) as a push notification provider.

  • DOWN sends an error level alert, UP sends success.
  • Test, certificate expiry and domain expiry alerts send info.
  • Needs an API key and a channel (default: general).

AI disclosure: the code was written with AI help (Claude and Codex). I reviewed it and tested it with real alerts on my phone.

Please follow this checklist to avoid unnecessary back and forth (click to expand)
  • ⚠️ If there are Breaking change (a fix or feature that alters existing functionality in a way that could cause issues) I have called them out
  • 🧠 I have disclosed any use of LLMs/AI in this contribution and reviewed all generated content.
    I understand that I am responsible for and able to explain every line of code I submit.
  • 🔍 Any UI changes adhere to visual style of this project.
  • 🛠️ I have self-reviewed and self-tested my code to ensure it works as expected.
  • 📝 I have commented my code, especially in hard-to-understand areas (e.g., using JSDoc for methods).
  • 🤖 I added or updated automated tests where appropriate.
  • 📄 Documentation updates are included (if applicable).
  • 🧰 Dependency updates are listed and explained.
  • ⚠️ CI passes and is green.

Screenshots for Visual Changes

Event Before After
UP N/A phone-up
DOWN N/A phone-down
Certificate-expiry N/A phone-cert-expired
Domain-expiry N/A phone-domain-expiry
Testing N/A phone-test

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR! If anyone would like to help with testing, run: npx kuma-pr hchouhan:feat/trigv-notification-provider (requires Node.js and Docker)

@flowdee

flowdee commented Nov 14, 2026

Copy link
Copy Markdown

+1 for that

This branch has not been deployed

No deployments
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