HomePhabricator
Diffusion Comm 19e5f0ff3d84

Send all notifications received by backgrounded app to JS at once

Description

Send all notifications received by backgrounded app to JS at once

Summary:
Previously for each notification received by backgrounded app we sent separate JS event so that it could be persisted in redux and SQLite. However there was
no reason not to send one JS event with array of all messages. It can potentially help the issue:
https://linear.app/comm/issue/ENG-5147/messageinfos-from-notifs-take-too-long-to-import-on-ios-start#comment-a2fb0774

Test Plan:

  1. Build iOS app.
  2. Background it.
  3. Send multiple messages.
  4. Kill the keyserver.
  5. Bring the app to the foreground again.
  6. Ensure all messages are available in the relevant thread(s) even the keyserver is not running.

Reviewers: tomek, bartek, michal

Reviewed By: tomek

Subscribers: ashoat, wyilio

Differential Revision: https://phab.comm.dev/D9429

Details

Provenance
marcinAuthored on Oct 9 2023, 9:15 AM
Reviewer
tomek
Differential Revision
D9429: Send all notifications received by backgrounded app to JS at once
Parents
rCOMMcf102e2a89b3: Revert "[native] Temporary changes for staff release"
Branches
Unknown
Tags
Unknown