Page MenuHomePhabricator

[lib][web] Introduce threadIDIsThick
ClosedPublic

Authored by ashoat on Feb 11 2025, 12:21 AM.
Tags
None
Referenced Files
F6012717: D14338.id47043.diff
Sat, Apr 19, 3:16 PM
Unknown Object (File)
Sat, Apr 19, 11:10 AM
Unknown Object (File)
Sat, Apr 19, 4:21 AM
Unknown Object (File)
Fri, Apr 18, 2:07 PM
Unknown Object (File)
Thu, Apr 17, 4:08 PM
Unknown Object (File)
Mon, Apr 7, 7:51 PM
Unknown Object (File)
Fri, Mar 28, 3:31 AM
Unknown Object (File)
Wed, Mar 26, 7:39 AM
Subscribers
None

Details

Summary

We already have threadTypeIsThick, but we don't have access to the ThreadType in getMessageNotifyType, and it would be expensive to add it.

Instead, we can determine if the message is in a thick thread by just checking the ID.

Depends on D13919

Test Plan

Flow and careful code inspection (this is a very simple change)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable