Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Slack] Include URL in slack alert #3351

Closed
1 task done
wojcikm opened this issue Jul 4, 2023 · 5 comments · Fixed by #3886
Closed
1 task done

[Slack] Include URL in slack alert #3351

wojcikm opened this issue Jul 4, 2023 · 5 comments · Fixed by #3886
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added type:enhance-existing feature wants to enhance existing monitor

Comments

@wojcikm
Copy link

wojcikm commented Jul 4, 2023

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

UI Feature

🔖 Feature description

URL in slack notification is missing now:
image

Which is cumbersome to check the website that is failing - I have to go to the dashboard and search for it directly.

✔️ Solution

Add URL anywhere in the message - it would be highly appreciated

❓ Alternatives

No response

📝 Additional Context

No response

@wojcikm wojcikm added the feature-request Request for new features to be added label Jul 4, 2023
@CommanderStorm
Copy link
Collaborator

what kind of monitor are you using?
Could this be another symptom of the issue discussed in #3274

@wojcikm
Copy link
Author

wojcikm commented Jul 5, 2023

It's HTTP(s) monitor @CommanderStorm

@CommanderStorm
Copy link
Collaborator

Adding a button like this

if (baseURL) {
data.attachments.forEach(element => {
element.blocks.push({
"type": "actions",
"elements": [{
"type": "button",
"text": {
"type": "plain_text",
"text": "Visit Uptime Kuma",
},
"value": "Uptime-Kuma",
"url": baseURL + getMonitorRelativeURL(monitorJSON.id),
}],
});
});
}

but pointing to this URL should be eazy:
"url": monitorJSON.url,

The problem is that I don't have a working notification setup for slack. Could you test if adding such a change works and provide a PR?

@mstgelais

This comment was marked as spam.

@CommanderStorm

This comment was marked as resolved.

DaanMeijer added a commit to september-digital-agency/uptime-kuma that referenced this issue Oct 13, 2023
@chakflying chakflying changed the title Include URL in slack alert [Slack] Include URL in slack alert Dec 4, 2023
@CommanderStorm CommanderStorm added the area:notifications Everything related to notifications label Dec 6, 2023
@CommanderStorm CommanderStorm added the type:enhance-existing feature wants to enhance existing monitor label Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added type:enhance-existing feature wants to enhance existing monitor
Projects
None yet
3 participants