Page MenuHomePhabricator

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

Authored by ashoat on Sep 30 2024, 9:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 11:53 AM
Unknown Object (File)
Wed, Dec 18, 11:53 AM
Unknown Object (File)
Wed, Dec 18, 11:52 AM
Unknown Object (File)
Wed, Dec 18, 11:52 AM
Unknown Object (File)
Mon, Dec 9, 9:53 AM
Unknown Object (File)
Tue, Nov 26, 4:43 AM
Unknown Object (File)
Mon, Nov 25, 5:13 AM
Unknown Object (File)
Sat, Nov 23, 3:21 AM
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