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

Stop notify about no notification every time #3851

Open
maxx-ukoo opened this issue Jan 28, 2025 · 4 comments
Open

Stop notify about no notification every time #3851

maxx-ukoo opened this issue Jan 28, 2025 · 4 comments
Labels
enhancement Indicates new feature requests

Comments

@maxx-ukoo
Copy link

Is your feature request related to a problem? Please describe.

On every start app do reminder about not alloved notification and block control panel with this notification.

Describe the solution you'd like

App should ask about enable notification only one time on first start.

Describe alternatives you've considered

Don't reminder about not enabled notification at all

Additional context

We should remove reminder about missing some features:
Image

@maxx-ukoo maxx-ukoo added the enhancement Indicates new feature requests label Jan 28, 2025
@mueller-ma
Copy link
Member

The easiest way to hide this message is to allow notifications, but I guess you don't want that. May I ask why don't you want to enable notifications?

@maxx-ukoo
Copy link
Author

@mueller-ma according to the principle of least privilege (POLP) app (in this context) should have minimum rights. I don't expect any notifications from app, as result notifications must be disabled.

@mueller-ma
Copy link
Member

according to the principle of least privilege (POLP) app (in this context) should have minimum rights

That's correct in general, but in this case the permission doesn't grant access to any data. It just allows sending notification. Also the user is notified when this permission is used is misused as a notification is shown then.

@maniac103 What do you think about this? Notifications are used for myopenhab push notifications, but also for errors on background tasks, which include voice widgets, sending device info and scanning NFC tags.
Maybe add a "Never remind" action to this snackbar?

@maniac103
Copy link
Contributor

Yeah, a 'Got it' button should be OK when combined with an info text in the prefs panel, which could have a button to re-request the permission grant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Indicates new feature requests
Projects
None yet
Development

No branches or pull requests

3 participants