Page MenuHomePhabricator

[lib] Disable text/media notif when starting sidebar
ClosedPublic

Authored by ashoat on Feb 20 2023, 5:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 20, 3:37 AM
Unknown Object (File)
Sat, Apr 19, 9:54 PM
Unknown Object (File)
Sat, Apr 19, 9:25 PM
Unknown Object (File)
Sat, Apr 19, 12:36 AM
Unknown Object (File)
Thu, Apr 17, 12:19 PM
Unknown Object (File)
Mon, Apr 14, 8:00 AM
Unknown Object (File)
Fri, Apr 11, 3:56 PM
Unknown Object (File)
Tue, Apr 8, 3:35 PM
Subscribers

Details

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)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable