Page MenuHomePhabricator

[lib][native][web] Set sidebarCreation flag when creating messages that triggered a sidebar
ClosedPublic

Authored by ashoat on Feb 20 2023, 5:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 16 2024, 2:05 AM
Unknown Object (File)
Nov 4 2024, 11:23 PM
Unknown Object (File)
Oct 29 2024, 4:20 PM
Unknown Object (File)
Oct 29 2024, 4:20 PM
Unknown Object (File)
Oct 29 2024, 4:20 PM
Unknown Object (File)
Oct 29 2024, 4:19 PM
Unknown Object (File)
Oct 28 2024, 1:49 PM
Unknown Object (File)
Oct 20 2024, 10:25 AM
Subscribers

Details

Summary

This is the final diff in the stack. We set sidebarCreation on the client, which makes it so notifs aren't sent by the keyserver.

Depends on D6799

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