HomePhabricator
Diffusion Comm 8bac0095a624

[lib] Simplify useExistingThreadInfoFinder

Description

[lib] Simplify useExistingThreadInfoFinder

Summary:
Since D13404, pendingToRealizedThreadIDsSelector will return the same thing when queried with a thin thread versus a thick thread.

As such, there's no longer a reason to check for both separately here.

Separately, I also got rid of the ternary at the end in favor of an early exit.

Depends on D13533

Test Plan: Careful reading of the code, as well as being tested with the rest of the stack, where I test all kinds of ways of creating a thread

Reviewers: tomek, kamil, inka

Reviewed By: inka

Differential Revision: https://phab.comm.dev/D13534

Details

Provenance
ashoatAuthored on Fri, Sep 27, 11:39 AM
Reviewer
inka
Differential Revision
D13534: [lib] Simplify useExistingThreadInfoFinder
Parents
rCOMM2d4e0c353da3: [lib] Update useUserProfileThreadInfo to consider thick threads
Branches
Unknown
Tags
Unknown