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

Added sound option for pushover #996

Merged
merged 1 commit into from
Oct 7, 2023

Conversation

szerencl
Copy link
Contributor

@szerencl szerencl commented Oct 6, 2023

Fixes #522

Added option to configure notification sound for pushover.
Also updated the documentation to include it and priority as well, because it was undocumented.

@szerencl

This comment was marked as resolved.

Copy link
Owner

@crazy-max crazy-max left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!

@crazy-max crazy-max merged commit 92c1a6a into crazy-max:master Oct 7, 2023
32 checks passed
@Circenn5130
Copy link

I'm getting the error below when using this environment variable in Kubernetes.
DIUN_NOTIF_PUSHOVER_SOUND

Exact syntax in .env
DIUN_NOTIF_PUSHOVER_SOUND=bugle

FTL Cannot load configuration error="failed to decode configuration from environment variables: Failed to decode configuration from environment variables: field not found, node: sound"

image

@crazy-max
Copy link
Owner

@Circenn5130 This feature has not been released yet.

@Circenn5130
Copy link

Thank you, and my apologies. I'm still not very familiar with GitHub.

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.

Pushover Notification Customization of sound and priority
3 participants