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

Combine and limit delete-notification pushes #936

Closed
2 tasks
nickvergessen opened this issue Apr 19, 2021 · 2 comments
Closed
2 tasks

Combine and limit delete-notification pushes #936

nickvergessen opened this issue Apr 19, 2021 · 2 comments

Comments

@nickvergessen
Copy link
Member

nickvergessen commented Apr 19, 2021

Currently all mobile clients always receive all delete-pushes. iOS ignores silent background notifications if there are too many, so we plan to limit and combine them:

  • Only send talk to talk apps and all others to the files app
  • Combine the multiple deletes into 1 push, so instead of:
   		[
   			'nid' => 13,
   			'delete' => true,
   		]
   		[
   			'nid' => 42,
   			'delete' => true,
   		]
   		[
   			'nid' => 23,
   			'delete' => true,
   		]

the Talk iOS app receives:

   		[
   			'nids' => [13, 42, 23],
   			'delete-multiple' => true,
   		]
@blizzz
Copy link
Member

blizzz commented May 3, 2022

moving to 25

@SystemKeeper
Copy link

Fixed by #1429 and included in 26.0.0 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants