Page MenuHomePhabricator

[lib] Make text and media message types read-only
ClosedPublic

Authored by ashoat on Feb 20 2023, 5:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 26 2024, 7:16 PM
Unknown Object (File)
Oct 29 2024, 4:19 PM
Unknown Object (File)
Oct 19 2024, 10:22 AM
Unknown Object (File)
Oct 18 2024, 3:27 AM
Unknown Object (File)
Oct 18 2024, 3:27 AM
Unknown Object (File)
Oct 18 2024, 3:27 AM
Unknown Object (File)
Sep 30 2024, 4:20 AM
Unknown Object (File)
Sep 30 2024, 4:20 AM
Subscribers

Details

Summary

Was updating these types and figured it would be good to make them read-only.

Test Plan

Flow. Also 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