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 ntfy Notifications via Webhooks #1579

Merged
merged 8 commits into from
Jul 18, 2024
Merged

Add ntfy Notifications via Webhooks #1579

merged 8 commits into from
Jul 18, 2024

Conversation

svenvg93
Copy link
Contributor

@svenvg93 svenvg93 commented Jun 26, 2024

πŸ“ƒ Description

To cross ntfy of the list #40

πŸͺ΅ Changelog

βž• Added

  • Add ntfy to the notification page
  • Add ntfy to the settings migration
  • Add a placeholder for the URL
  • add field username with placeholder
  • add field password with placeholder
  • Add a field for the topic with placeholder
  • make the username and password optional

πŸ“· Screenshots

SchermΒ­afbeelding 2024-06-26 om 22 02 32 Schermafbeelding_2024-06-26_om_22 52 32

@svenvg93 svenvg93 marked this pull request as ready for review June 26, 2024 21:00
@svenvg93 svenvg93 requested a review from alexjustesen as a code owner June 26, 2024 21:00
@svenvg93
Copy link
Contributor Author

svenvg93 commented Jun 26, 2024

@alexjustesen let me know if you want me to bundle these all in one pull :)
#1579
#1578
#1574
#1567

@alexjustesen alexjustesen added the πŸŽ‰ feature New feature or request label Jul 18, 2024
@alexjustesen alexjustesen merged commit ac8c709 into alexjustesen:develop-0.21.0 Jul 18, 2024
1 check passed
@svenvg93 svenvg93 deleted the 40-add-ntfy branch July 18, 2024 13:56
@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
πŸŽ‰ feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants