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.

Sign up for GitHub

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

Jump to bottom

Calendar-only notifications #7149

Open
2 of 8 tasks
murilopereirame opened this issue Jul 1, 2024 · 0 comments · May be fixed by #7181
Open
2 of 8 tasks

Calendar-only notifications #7149

murilopereirame opened this issue Jul 1, 2024 · 0 comments · May be fixed by #7181
Assignees
@murilopereirame
Labels
goal: best calendar app topic: calendar
Milestone
Calendar App

Comments

@murilopereirame
Copy link
Contributor

murilopereirame commented Jul 1, 2024 •
edited
Loading

AC

  • Calendar app should only receive calendar related notifications
  • There is a new push identifier for the calendar app

Tasks

  • Set up a new push identifier
  • Make sure tutadb handles the new push identifier

Test Notes

  • Calendar app produces a new push identifier different from Mail
  • Preexisting push identifiers still working in the mail app (Just for mail notifications)
  • Calendar app doesn't receive mail notifications
  • Calendar app receives alarm notifications
  • Mail app doesn't receive alarm notifications
  • Mail app receives mail notifications

Pre-release tasks

  • Merge MR 641
@murilopereirame murilopereirame added this to the Calendar App milestone Jul 1, 2024
@murilopereirame murilopereirame self-assigned this Jul 8, 2024
murilopereirame pushed a commit that referenced this issue Jul 8, 2024
[Calendar] Adds differentiation between push identifiers
f5fff28 
This commit adds the App field to push identifiers, letting the
server know if the identifier belongs to Mail or Calendar app and
send notifications correctly according to the app.

Closes #7149
@murilopereirame murilopereirame linked a pull request Jul 8, 2024 that will close this issue
Open
@murilopereirame murilopereirame linked a pull request Jul 8, 2024 that will close this issue
[Calendar] Calendar Notifications #7181
Open
murilopereirame pushed a commit that referenced this issue Jul 8, 2024
[Calendar] Adds differentiation between push identifiers
3e33aab 
This commit adds the App field to push identifiers, letting the
server know if the identifier belongs to Mail or Calendar app and
send notifications correctly according to the app.

Closes #7149
murilopereirame pushed a commit that referenced this issue Jul 8, 2024
[Calendar] Adds differentiation between push identifiers
42660e8 
This commit adds the App field to push identifiers, letting the
server know if the identifier belongs to Mail or Calendar app and
send notifications correctly according to the app.

Closes #7149
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@murilopereirame murilopereirame

Labels
goal: best calendar app topic: calendar
Projects
None yet
Milestone
Calendar App
Development

Successfully merging a pull request may close this issue.

[Calendar] Calendar Notifications tutao/tutanota
1 participant
@murilopereirame