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

Add Setting in Mail App to Disable Calendar Notifications #7536

Closed
4 tasks done
wrdhub opened this issue Sep 6, 2024 · 0 comments · Fixed by #7558
Closed
4 tasks done

Add Setting in Mail App to Disable Calendar Notifications #7536

wrdhub opened this issue Sep 6, 2024 · 0 comments · Fixed by #7558
Assignees
Labels
state:tested We tested it and are about to release it
Milestone

Comments

@wrdhub
Copy link
Contributor

wrdhub commented Sep 6, 2024

User story: I don't want double notifications for calendar events if I have the Calendar and mail app installed.

AC:

  • A per user, per device setting on the mail app
  • "Receive calendar event notifications?" → a yes/no toggle
  • When this setting is "no", if notifications are active for that app, mail notifications are received but calendar notifications are not
  • When this setting is "yes", if notifications are active for that app, mail notifications and calendar notifications are received
  • When notifications are not active for that app, this option shows but is grayed out (and set to no)
  • When this setting is changed to "no", calendar event notifications that have been scheduled should be unscheduled and no future events should be notified.
  • When this setting is changed to "yes", calendar events with existing notifications on them should be rescheduled and send when applicable.

Test notes

On Mail mobile app:

  • In the settings page there's a new select field to control if Mail app should receive calendar notifications/alarms.
  • If the settings is active nothing should change, and all notifications should arrive.
  • If it is disabled, only email notifications should appear for the mail app.
  • Calendar app, Desktop client and Webapp shouldn't be affected by this setting.
@andrehgdias andrehgdias self-assigned this Sep 9, 2024
@murilopereirame murilopereirame linked a pull request Sep 19, 2024 that will close this issue
@murilopereirame murilopereirame added this to the Calendar App Release milestone Sep 24, 2024
@murilopereirame murilopereirame added the state:tested We tested it and are about to release it label Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:tested We tested it and are about to release it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants