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

Allow finer control over UnifiedPush in the android app #646

Closed
ollien opened this issue Mar 2, 2023 · 5 comments
Closed

Allow finer control over UnifiedPush in the android app #646

ollien opened this issue Mar 2, 2023 · 5 comments
Labels
android-app ntfy Android app enhancement New feature or request unified-push UnifiedPush feature or bug

Comments

@ollien
Copy link
Contributor

ollien commented Mar 2, 2023

Right now, if an app wants to use UnifiedPush, it can simply register itself. This causes ntfy to use instant delivery, which makes sense given the usecase of UnifiedPush, but is a bit frustrating if you only wanted to use ntfy for other pushes and don't want the battery drain of instant delivery.

Ideas for possible controls:

  1. An option to disable UnifiedPush entirely
  2. An option to block a specific app from enabling UnifiedPush
  3. An option to disable instant delivery for a UnifiedPush topic

I don't know how the project feels about any of these, but I imagine that #1 is perhaps the "easiest" (said without much context).

@binwiederhier
Copy link
Owner

(I am assuming that you are talking about the Android app)

I used to have a toggle in the Settings that allowed disabling UnifiedPush entirely, but I removed that because I didn't want toggle-hell. I haven't heard this argument before, to be honest. Usually you can choose the notification method in the app itself and unselect "ntfy" there. What app is misbehaving and registering itself without your consent?

@binwiederhier binwiederhier added enhancement New feature or request android-app ntfy Android app labels Mar 3, 2023
@binwiederhier
Copy link
Owner

/cc @S1m @karmanyaahm

@binwiederhier binwiederhier added the unified-push UnifiedPush feature or bug label Mar 3, 2023
@ollien
Copy link
Contributor Author

ollien commented Mar 3, 2023

Yeah, I am.

The offending app is Tusky. As far as I can tell there's no option for it. See this issue tuskyapp/Tusky#2692

@binwiederhier
Copy link
Owner

binwiederhier commented Mar 3, 2023

I'd be ok with an off switch for UP if somebody wants to implement it (cough, @wunter8, maybe?). That said, it does seem like a Tusky issue.

@binwiederhier
Copy link
Owner

@wunter8 Did this already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android-app ntfy Android app enhancement New feature or request unified-push UnifiedPush feature or bug
Projects
None yet
Development

No branches or pull requests

2 participants