Page MenuHomePhabricator

[web/native] Prevent text/multimedia from being pinned if they begin a sidebar
ClosedPublic

Authored by rohan on Oct 19 2023, 12:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 11, 2:50 PM
Unknown Object (File)
Fri, Apr 11, 12:35 PM
Unknown Object (File)
Fri, Apr 11, 8:06 AM
Unknown Object (File)
Thu, Apr 10, 6:34 AM
Unknown Object (File)
Wed, Apr 9, 7:44 AM
Unknown Object (File)
Wed, Mar 26, 11:16 AM
Unknown Object (File)
Wed, Mar 26, 11:16 AM
Unknown Object (File)
Mon, Mar 24, 2:18 PM
Subscribers

Details

Summary

In the same way we handle reactions, we should prevent users from pinning messages (specifically text/multimedia) within a sidebar if they are the first message in the sidebar. We'll still allow other messages to be pinned in the sidebar.

Resolves https://linear.app/comm/issue/ENG-4847/not-possible-to-pin-the-source-messge

Depends on D9543

Test Plan

Confirmed that pinning messages in both a top-level thread and a sidebar still work as expected, and confirmed that the pin option doesn't appear for the first message in a sidebar on both web and native

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable