HomePhabricator
Diffusion Comm b0b5ac75c171

[lib] Disable text/media notif when starting sidebar

Description

[lib] Disable text/media notif when starting sidebar

Summary:
More context in ENG-3041. Today we send two notifs when a sidebar is created; this diff makes it so we only send one notif.

Depends on D6797

Test Plan:
I tested this whole stack as follows:

  1. Create a sidebar by sending a text message on web
  2. Create a sidebar by sending a media message on web
  3. Create a sidebar by sending a text message on native
  4. Create a sidebar by sending a media message on native

In all of these cases, I confirmed that only one notif was delivered (the notif for the text/media message was skipped)

Reviewers: atul

Reviewed By: atul

Subscribers: tomek

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

Details

Provenance
ashoatAuthored on Feb 19 2023, 2:09 PM
Reviewer
atul
Differential Revision
D6798: [lib] Disable text/media notif when starting sidebar
Parents
rCOMMc36f710f648c: [keyserver][lib] Pass MessageData to generatesNotifs
Branches
Unknown
Tags
Unknown