Page MenuHomePhabricator

[web] Only call userInfoSelectorForPotentialMembers where it's needed
ClosedPublic

Authored by ashoat on Mon, Sep 30, 9:54 PM.
Tags
None
Referenced Files
F2860545: D13541.id44771.diff
Tue, Oct 1, 6:12 PM
F2860099: D13541.id.diff
Tue, Oct 1, 3:47 PM
F2853633: D13541.id44722.diff
Mon, Sep 30, 10:29 PM
F2853544: D13541.id.diff
Mon, Sep 30, 10:27 PM
F2853379: D13541.diff
Mon, Sep 30, 10:11 PM
Subscribers
None

Details

Summary

The only callsite of useInfosForPendingThread that needs otherUserInfos is ChatMessageListContainer. It's only needed there to pass to ChatThreadComposer.

To avoid fetching it unnecessarily at the other callsites, I've extracted it to that single callsite. This also benefits code readability by reducing indirection and unnecessary prop drilling.

Depends on D13540

Test Plan

Flow (this is a pure refactor)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable