Skip to content

Conversation

@justinjakull
Copy link

Added rich "Uptime Kuma"-like Discord embed messages to Peekaping.
Fixes #149
Both Ping and Time are conditional and only displayed if available.
I added additional bindings to make all of the features below also accessible to other notification services

Screenshot 2025-10-16 at 18 47 47

NOTE: I can't do frontend (I can barely do backend, this is my first go Project), so someone else would need to add the hints for the new bindings.

@codecov
Copy link

codecov bot commented Oct 16, 2025

Codecov Report

❌ Patch coverage is 0% with 81 lines in your changes missing coverage. Please review.
✅ Project coverage is 19.82%. Comparing base (12cb83e) to head (263c2b4).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
.../modules/notification_channel/providers/discord.go 0.00% 52 Missing ⚠️
...c/modules/notification_channel/providers/common.go 0.00% 29 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #216      +/-   ##
==========================================
- Coverage   19.90%   19.82%   -0.09%     
==========================================
  Files         181      181              
  Lines       18829    18907      +78     
==========================================
  Hits         3748     3748              
- Misses      14897    14975      +78     
  Partials      184      184              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@justinjakull
Copy link
Author

Sorry, forgot to pull main and added Forum support. Note: I would like to add tags as well, not sure where to find the appropriate bindings though.

@Neshura87
Copy link

Would be nice if we could get this merged, it's the one gripe I currently have with peekaping

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.

Discord notification is missing monitor name

2 participants