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 support for choosing a priority level for notifications (ntfy) #553

Closed
1 task done
Eirikr70 opened this issue Feb 2, 2024 · 2 comments
Closed
1 task done
Labels
Feature request➕ New feature or request next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed.

Comments

@Eirikr70
Copy link

Eirikr70 commented Feb 2, 2024

Is there an existing issue for this?

  • I have searched the existing open and closed issues

Is your feature request related to a problem? Please describe

I have found no way to configure the priorities of the notifications emitted by pi.alert to ntfy. They seem to always be set at the highest level. It would be nice to be able to define the priority level.

Describe the solution you'd like

Permit the configuration of the alert priority by inserting a Priority field in the ntfy (and the other) chapter of the settings.

Describe alternatives you've considered

I haven't found any way to set up the priority in the configuration on the ntfy side.

Anything else?

@Eirikr70 Eirikr70 added the Feature request➕ New feature or request label Feb 2, 2024
jokob-sk added a commit that referenced this issue Feb 2, 2024
@jokob-sk jokob-sk added the next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed. label Feb 2, 2024
@jokob-sk
Copy link
Owner

jokob-sk commented Feb 2, 2024

Hi @Eirikr70 ,

This should be available in the next release. Please test the setting in the pi.alert_dev image after this run finishes (https://github.com/jokob-sk/Pi.Alert/actions/runs/7761362970) if you have a minute.

Thanks,
j

@jokob-sk
Copy link
Owner

released -> closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request➕ New feature or request next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed.
Projects
None yet
Development

No branches or pull requests

2 participants