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

Cannot take off alarm without adding notification permission #220

Open
7 tasks done
austinoneil opened this issue Apr 28, 2024 · 0 comments · May be fixed by #221
Open
7 tasks done

Cannot take off alarm without adding notification permission #220

austinoneil opened this issue Apr 28, 2024 · 0 comments · May be fixed by #221
Labels
enhancement New feature or request

Comments

@austinoneil
Copy link

Checklist

  • I can reproduce the bug with the latest version given here.
  • 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 bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.0.3

Affected Android/Custom ROM version

GrapheneOS latest

Affected device model

Pixel 7

How did you install the app?

F-Droid / IzzyOnDroid

Steps to reproduce the bug

  1. Make sure app does not have notification permission.
  2. Attempt to add an event.
  3. Touch the reminder icon, attempting to remove it. It says "10 minutes before" by default.

Expected behavior

Modal shows, allowing user to remove the reminder.

Actual behavior

App requests notification permission and cannot proceed until granted permission.

Screenshots/Screen recordings

screen-20240428-114152.mp4
screen-20240428-114447.mp4

Additional information

Suggested fix: If permission is not already granted, set the default alarm to be "No reminder." If the user opens the notification dialog, prompt user for permission.

@austinoneil austinoneil added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Apr 28, 2024
@Aga-C Aga-C added enhancement New feature or request and removed bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Apr 28, 2024
@austinoneil austinoneil linked a pull request Apr 28, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants