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
F5186957: D6798.id.diff
Fri, Apr 4, 4:51 PM
Unknown Object (File)
Thu, Apr 3, 1:25 PM
Unknown Object (File)
Wed, Apr 2, 3:03 PM
Unknown Object (File)
Sat, Mar 8, 5:01 AM
Unknown Object (File)
Feb 25 2025, 5:28 PM
Unknown Object (File)
Feb 25 2025, 5:28 PM
Unknown Object (File)
Feb 14 2025, 6:23 PM
Unknown Object (File)
Jan 26 2025, 12:09 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