[Discord] Service URL should only be included if defined #3327
Labels
area:notifications
Everything related to notifications
feature-request
Request for new features to be added
good first issue
Good for newcomers
type:enhance-existing
feature wants to enhance existing monitor
🛡️ Security Policy
Description
This is what I received from the Discord notification webhook when one of the child services of a group I created went down. It display a section
Service URL
with a valuehttps://
. I don't think this section is necessary since it's a group and there's no related configuration in the settings.The similar situation happened when the service is up. The section
Service URL
displayhttps://
. And thePing
section displayN/A
.👟 Reproduction steps
Set up a Discord notification and enable it in a group.
👀 Expected behavior
Service URL
section should maintain empty since there's nothing to display with the URL.Ping
section should be remove or replace with other informations.😓 Actual Behavior
The notification display unnecessary informations.
Service URL
display a valuehttps://
.Ping
display a valueN/A
.🐻 Uptime-Kuma Version
1.22.0
💻 Operating System and Arch
Ubuntu 22.04.2
🌐 Browser
No browser, it happens in Discord.
🐋 Docker Version
No response
🟩 NodeJS Version
16.20.1
📝 Relevant log output
No response
The text was updated successfully, but these errors were encountered: