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)
Wed, Nov 20, 4:38 PM
Unknown Object (File)
Wed, Nov 20, 4:38 PM
Unknown Object (File)
Wed, Nov 20, 4:37 PM
Unknown Object (File)
Wed, Nov 20, 4:18 PM
Unknown Object (File)
Tue, Nov 19, 11:57 PM
Unknown Object (File)
Oct 19 2024, 4:19 AM
Unknown Object (File)
Oct 19 2024, 4:19 AM
Unknown Object (File)
Oct 19 2024, 4:18 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