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

[Feature] Configurable Gotify priority #1277

Closed
gandy92 opened this issue Jan 12, 2022 · 3 comments · Fixed by #1279
Closed

[Feature] Configurable Gotify priority #1277

gandy92 opened this issue Jan 12, 2022 · 3 comments · Fixed by #1279
Labels
joex affects the joex component webui WebUI issues

Comments

@gandy92
Copy link

gandy92 commented Jan 12, 2022

Currently, when an event is pushed to Gotify, the notification priority appears to be 0. The Gotify android app uses the message priority to decide how to present the message in the notification bar (see https://github.com/gotify/android#message-priorities). Priority 0 leads to the least noticeable notification possible.
My feature-request entails adding the message priority (0-10) to the Gotify Webhook configuration and using the configured priority for that specific webhook. In use-cases where the message priority must depend on the type of event, multiple webhooks can be used.

@eikek
Copy link
Owner

eikek commented Jan 12, 2022

Ah right, thank you!

@eikek eikek added joex affects the joex component webui WebUI issues labels Jan 12, 2022
@eikek eikek added this to the Docspell 0.31.0 milestone Jan 12, 2022
eikek added a commit that referenced this issue Jan 12, 2022
@mergify mergify bot closed this as completed in #1279 Jan 12, 2022
@gandy92
Copy link
Author

gandy92 commented Jan 13, 2022

Cool, thank you for accepting the request - and even implementing it right away, I'll be pulling the new release right away

@eikek
Copy link
Owner

eikek commented Jan 13, 2022

You're welcome! I wanted to add it from the beginning but forgot about it - and then was wondering why my notifications were so shy on the phone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
joex affects the joex component webui WebUI issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants