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

Set "Permission Required" popup to only appear once, thnk u #284

Closed
6 of 7 tasks
TheAlmightyOgreLord opened this issue Jan 17, 2025 · 1 comment
Closed
6 of 7 tasks
Labels
invalid This does not seem right

Comments

@TheAlmightyOgreLord
Copy link

TheAlmightyOgreLord commented Jan 17, 2025

Checklist

  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the request will be dismissed otherwise.
  • This issue contains only one feature request.
  • I have read and understood the contribution guidelines.
  • I optionally donated to support the Fossify mission.

Feature description

Disable annoying persistence of "Permission required" notification when disabling notification permission for the app to enhance compatability with SpamBlocker and other DNC/stir/shaken apps.

My current fix is to use DND mode as a system-wide per-app notification filter with Messages notifications set to enabled (to stop the notification popping up), with SpamBlocker notifications also set to enabled, then on DND I have Messages notifications set to disabled. This prevents double messages but gets rid of the popup.

Why do you want this feature?

Greater compatability with DNC spam blocking apps that enable stir/shaken functionality on devices. Also the stock sms app (with the ugly dark mode) does not exhibit this behavior. It only prompts to enable notification permission once then never again.

Additional information

Image
@TheAlmightyOgreLord TheAlmightyOgreLord added feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Jan 17, 2025
@Aga-C
Copy link
Member

Aga-C commented Jan 18, 2025

In notification settings, disable Received SMS channel instead of revoking a permission.

Image

We don't want to disable this pop-up, because users like clicking through things without reading, and then they will be surprised that they don't get any notifications.

@Aga-C Aga-C closed this as not planned Won't fix, can't repro, duplicate, stale Jan 18, 2025
@Aga-C Aga-C added invalid This does not seem right and removed feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This does not seem right
Projects
None yet
Development

No branches or pull requests

2 participants