mirror of
https://github.com/healthchecks/healthchecks.git
synced 2026-04-25 06:55:53 +03:00
[PR #12] [MERGED] Add PushOver integration #860
Labels
No labels
bug
bug
bug
feature
good-first-issue
new integration
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/healthchecks#860
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 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:
master← Head:pushover📝 Commits (4)
85c1f65Add Pushover integration280bd6aHandle Pushover emergency notifications6c9e3bbMerge branch 'master' into pushover99cb654Fix 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.valuefield, they are stored as pipe-separated values inside a string. It's a little hacky, but works fine.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.