-
-
Notifications
You must be signed in to change notification settings - Fork 203
feat: Apprise notification channel #2415
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
feat: Apprise notification channel #2415
Conversation
Removed commented code for InfluxDB write condition.
| public function up(): void | ||
| { | ||
| $this->migrator->add('notification.apprise_enabled', false); | ||
| $this->migrator->add('notification.apprise_sidecar_url', null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's name this just apprise_server_url.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
lang/en/settings/notifications.php
Outdated
| // Apprise notifications | ||
| 'apprise' => 'Apprise', | ||
| 'enable_apprise_notifications' => 'Enable Apprise notifications', | ||
| 'apprise_sidecar' => 'Apprise Sidecar', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's drop the "sidecar" and use "server" instead where applicable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
|
|
||
| public bool $apprise_enabled; | ||
|
|
||
| public ?string $apprise_sidecar_url; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apprise_server_url
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
📃 Description
This PR adds Apprise support with Filament v4, similar to #2380
closes #1961
closes #1620
Documentation
alexjustesen/speedtest-tracker-docs#94
Testing
To test Apprise you need to run local instance of Apprise or use the instance in the
docker-compose.ymlfor development.APPRISE_URLenv with the URL to your instance