[PR #12] [MERGED] Add PushOver integration #860

Closed
opened 2026-02-25 23:43:57 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/healthchecks/healthchecks/pull/12
Author: @Schnouki
Created: 11/17/2015
Status: Merged
Merged: 12/2/2015
Merged by: @cuu508

Base: masterHead: pushover


📝 Commits (4)

  • 85c1f65 Add Pushover integration
  • 280bd6a Handle Pushover emergency notifications
  • 6c9e3bb Merge branch 'master' into pushover
  • 99cb654 Fix tests when Pushover is not configured

📊 Changes

14 files changed (+248 additions, -10 deletions)

View changed files

📝 README.md (+11 -0)
📝 hc/api/admin.py (+2 -0)
hc/api/migrations/0017_auto_20151117_1032.py (+19 -0)
📝 hc/api/models.py (+39 -1)
📝 hc/front/tests/test_add_channel.py (+5 -1)
📝 hc/front/urls.py (+1 -0)
📝 hc/front/views.py (+67 -6)
📝 hc/settings.py (+6 -0)
📝 static/css/channels.css (+4 -0)
static/img/integrations/pushover.png (+0 -0)
📝 templates/front/channel_checks.html (+1 -1)
📝 templates/front/channels.html (+18 -1)
templates/integrations/add_pushover.html (+61 -0)
templates/integrations/pushover_message.html (+14 -0)

📄 Description

This adds PushOver integration.

PushOver is a service that allows to push notifications to Android and iOS devices (and desktop computers through a web browser addon) using a simple API.

PO "applications" have a free quota of 7,500 messages per month. So the best way to integrate PO is to let the user create an application (it's free and takes less than 2 minutes), and then use this to send notifications.
Because of this, there are at least 2 things to store for the integration to work: the app API token, and the user key. I also like to configure the priority of messages sent with PushOver (an integer between -2 and 2).
In order to store these 3 values in the Check.value field, they are stored as pipe-separated values inside a string. It's a little hacky, but works fine.

Sample notification on my phone

Everything seems to work find; however please tell me if anything needs to be improved before you can merge it. (Or if you just don't like it and don't want to merge it -- that's fine, I mostly did it for fun 😃)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/healthchecks/healthchecks/pull/12 **Author:** [@Schnouki](https://github.com/Schnouki) **Created:** 11/17/2015 **Status:** ✅ Merged **Merged:** 12/2/2015 **Merged by:** [@cuu508](https://github.com/cuu508) **Base:** `master` ← **Head:** `pushover` --- ### 📝 Commits (4) - [`85c1f65`](https://github.com/healthchecks/healthchecks/commit/85c1f65887e74f69a0d36e1d8640f991244b9263) Add Pushover integration - [`280bd6a`](https://github.com/healthchecks/healthchecks/commit/280bd6a2a244ac5d273cba8a006e73c68fc25093) Handle Pushover emergency notifications - [`6c9e3bb`](https://github.com/healthchecks/healthchecks/commit/6c9e3bb24d9f6e1de48d99d94f5d6dd011cdf633) Merge branch 'master' into pushover - [`99cb654`](https://github.com/healthchecks/healthchecks/commit/99cb654ec5730f6be33bb091aa3ac9e70963470c) Fix tests when Pushover is not configured ### 📊 Changes **14 files changed** (+248 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+11 -0) 📝 `hc/api/admin.py` (+2 -0) ➕ `hc/api/migrations/0017_auto_20151117_1032.py` (+19 -0) 📝 `hc/api/models.py` (+39 -1) 📝 `hc/front/tests/test_add_channel.py` (+5 -1) 📝 `hc/front/urls.py` (+1 -0) 📝 `hc/front/views.py` (+67 -6) 📝 `hc/settings.py` (+6 -0) 📝 `static/css/channels.css` (+4 -0) ➕ `static/img/integrations/pushover.png` (+0 -0) 📝 `templates/front/channel_checks.html` (+1 -1) 📝 `templates/front/channels.html` (+18 -1) ➕ `templates/integrations/add_pushover.html` (+61 -0) ➕ `templates/integrations/pushover_message.html` (+14 -0) </details> ### 📄 Description This adds PushOver integration. [PushOver](https://pushover.net) is a service that allows to push notifications to Android and iOS devices (and desktop computers through a web browser addon) using a simple API. PO "applications" have a free quota of 7,500 messages per month. So the best way to integrate PO is to let the user create an application (it's free and takes less than 2 minutes), and then use this to send notifications. Because of this, there are at least 2 things to store for the integration to work: the app API token, and the user key. I also like to configure the priority of messages sent with PushOver (an integer between -2 and 2). In order to store these 3 values in the `Check.value` field, they are stored as pipe-separated values inside a string. It's a little hacky, but works fine. ![Sample notification on my phone](https://cloud.githubusercontent.com/assets/64833/11214458/9ed012dc-8d41-11e5-868c-73fe10709f23.png) Everything seems to work find; however please tell me if anything needs to be improved before you can merge it. (Or if you just don't like it and don't want to merge it -- that's fine, I mostly did it for fun :smiley:) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-25 23:43:57 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/healthchecks#860
No description provided.