[web] Fetch loggedInUserInfo in ThreadListProvider
ClosedPublic
Actions

Authored by ashoat on Feb 17 2023, 2:54 PM.

Details

Reviewers

atul

Commits

rCOMM9906d6c36bbd: [web] Fetch loggedInUserInfo in ThreadListProvider

Summary

This diff should be a no-op. In a later diff, we'll need to pass loggedInUserInfo directly to getThreadListSearchResults.

I also introduce a small hook here: useLoggedInUserInfo.

Test Plan

Flow, and playing around with inbox on web

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Feb 17 2023, 2:54 PM

Herald added a subscriber: tomek. · View Herald TranscriptFeb 17 2023, 2:54 PM

ashoat added a child revision: D6754: [native] Add loggedInUserInfo prop to ChatThreadList.Feb 17 2023, 2:55 PM

Harbormaster completed remote builds in B16570: Diff 22671.Feb 17 2023, 3:10 PM

ashoat requested review of this revision.Feb 17 2023, 3:10 PM

atul accepted this revision.Feb 20 2023, 8:49 PM

This revision is now accepted and ready to land.Feb 20 2023, 8:49 PM

Closed by commit rCOMM9906d6c36bbd: [web] Fetch loggedInUserInfo in ThreadListProvider. · Explain WhyFeb 20 2023, 11:13 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM9906d6c36bbd: [web] Fetch loggedInUserInfo in ThreadListProvider.

Revision Contents
Changeset List

Path

Size

lib/

hooks/

account-hooks.js

14 lines

web/

chat/

thread-list-provider.js

6 lines

Diff 22762

View Options

lib/hooks/account-hooks.js