Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: send to multiple SMS numbers separated by comma #259

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

titanism
Copy link
Contributor

Similar to

const chatIds = getSecret("NOTIFICATION_TELEGRAM_CHAT_ID")?.split(",") ?? [];
for (const chatId of chatIds) {
await axios.post(
`https://api.telegram.org/bot${getSecret("NOTIFICATION_TELEGRAM_BOT_KEY")}/sendMessage`,
{
parse_mode: "Markdown",
disable_web_page_preview: true,
chat_id: chatId.trim(),
text: message.replace(/_/g, '\\_'),
}
);
}
we added support for SMS sending to multiple numbers.

@titanism
Copy link
Contributor Author

@AnandChowdhary bump

@AnandChowdhary AnandChowdhary merged commit 7766780 into upptime:master Feb 26, 2025
@titanism titanism deleted the patch-2 branch February 26, 2025 10:22
@titanism
Copy link
Contributor Author

@AnandChowdhary it doesn't look like npm release is working, can you check?

@AnandChowdhary
Copy link
Member

Making sure it's out now!

@upptime-bot
Copy link
Member

🎉 This PR is included in version 1.39.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@titanism
Copy link
Contributor Author

You're the best 🙏

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants