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)
Sun, Mar 16, 12:52 AM
Unknown Object (File)
Tue, Feb 25, 8:27 PM
Unknown Object (File)
Tue, Feb 25, 8:27 PM
Unknown Object (File)
Tue, Feb 25, 8:26 PM
Unknown Object (File)
Tue, Feb 25, 8:04 PM
Unknown Object (File)
Jan 25 2025, 1:11 AM
Unknown Object (File)
Jan 25 2025, 1:11 AM
Unknown Object (File)
Jan 25 2025, 1:09 AM
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