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

Add Pushover Notifications via Webhooks #1574

Merged
merged 5 commits into from
Jul 18, 2024
Merged

Add Pushover Notifications via Webhooks #1574

merged 5 commits into from
Jul 18, 2024

Conversation

svenvg93
Copy link
Contributor

πŸ“ƒ Description

To cross Pushover of the list #40

πŸͺ΅ Changelog

βž• Added

  • Add Pushover to the notification page
  • Add helperText to the User Key, API Token and url
  • Add Pushover to the settings migration
  • Add User Key and API Token to the input field

πŸ“· Screenshots

Settings;
SchermΒ­afbeelding 2024-06-25 om 23 50 58

Pushover:
SchermΒ­afbeelding 2024-06-25 om 23 44 42

@svenvg93 svenvg93 marked this pull request as ready for review June 25, 2024 22:00
@svenvg93
Copy link
Contributor Author

svenvg93 commented Jun 25, 2024

@alexjustesen The URL to sent the nottification to is always the same for every user. Fow now I left to be filled in by the user in case it ever changes on the Pushover side. But I can make it hardcode and remove that field if you prefer that.

@alexjustesen alexjustesen merged commit 2830aad into alexjustesen:develop-0.21.0 Jul 18, 2024
1 check passed
@svenvg93 svenvg93 deleted the 40-add-pushover branch July 18, 2024 13:51
@alexjustesen alexjustesen mentioned this pull request Jul 18, 2024
12 tasks
@alexjustesen alexjustesen added this to the v0.x.0 (notifications) milestone Jul 18, 2024
alexjustesen added a commit that referenced this pull request Aug 8, 2024
* Add Placeholder text for notifications (#1570)

first_commit

* Add Gotify Notifications via Webhook (#1561)

* first_commit

* Ready!

* Add_placeholder

* change url

---------

Co-authored-by: Alex Justesen <[email protected]>

* Add Support for Slack Notifications via Webhook (#1522)

* First Commit

* Fix lint

* add_url_placeholder

* Fix the liniting

* Add HealthCheck.io Notifications via Webhooks (#1567)

* first_commit

* linting

* add_descrip_for_threshold

* Change name

* add_url_placeholder

* fix_linting

* Add Pushover Notifications via Webhooks (#1574)

* add_pushover

* add_placeholder

* Linting

---------

Co-authored-by: Alex Justesen <[email protected]>

* Add ntfy Notifications via Webhooks (#1579)

* first_push

* Fix_json_payload

* Add_auth_option

* fix lint

* fix packet_loss_%

* added eof line

---------

Co-authored-by: Alex Justesen <[email protected]>

* Add Ookla URL to the notification (#1615)

* first commit

* added eof line

---------

Co-authored-by: Alex Justesen <[email protected]>

* Fix notifications layouts (#1639)

first commit

---------

Co-authored-by: Sven van Ginkel <[email protected]>
alexjustesen added a commit that referenced this pull request Aug 8, 2024
* Develop v0.21.0 (additional notification channels) (#1647)

* Add Placeholder text for notifications (#1570)

first_commit

* Add Gotify Notifications via Webhook (#1561)

* first_commit

* Ready!

* Add_placeholder

* change url

---------

Co-authored-by: Alex Justesen <[email protected]>

* Add Support for Slack Notifications via Webhook (#1522)

* First Commit

* Fix lint

* add_url_placeholder

* Fix the liniting

* Add HealthCheck.io Notifications via Webhooks (#1567)

* first_commit

* linting

* add_descrip_for_threshold

* Change name

* add_url_placeholder

* fix_linting

* Add Pushover Notifications via Webhooks (#1574)

* add_pushover

* add_placeholder

* Linting

---------

Co-authored-by: Alex Justesen <[email protected]>

* Add ntfy Notifications via Webhooks (#1579)

* first_push

* Fix_json_payload

* Add_auth_option

* fix lint

* fix packet_loss_%

* added eof line

---------

Co-authored-by: Alex Justesen <[email protected]>

* Add Ookla URL to the notification (#1615)

* first commit

* added eof line

---------

Co-authored-by: Alex Justesen <[email protected]>

* Fix notifications layouts (#1639)

first commit

---------

Co-authored-by: Sven van Ginkel <[email protected]>

* updated npm dependencies

---------

Co-authored-by: Sven van Ginkel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants