[lib] Inline createChatThreadItem
Needs ReviewPublic
Actions

Authored by ashoat on Wed, Oct 30, 11:42 AM.

Tags

None

Referenced Files

	F3105133: D13824.id45474.diff
	Wed, Oct 30, 8:29 PM

	F3104739: D13824.diff
	Wed, Oct 30, 6:29 PM

	F3104462: D13824.id45474.diff
	Wed, Oct 30, 5:18 PM

	F3104431: D13824.id.diff
	Wed, Oct 30, 5:17 PM

	F3104400: D13824.diff
	Wed, Oct 30, 5:16 PM

Subscribers

None

Details

Reviewers

tomek

Summary

Now that it's only called from one place, we can reduce the indirection by inlining it.

Depends on D13823

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Wed, Oct 30, 11:42 AM

ashoat added a child revision: D13825: [native] Handle rendering "No messages" inside MessagePreview.

Harbormaster completed remote builds in B32441: Diff 45474.Wed, Oct 30, 12:16 PM

ashoat requested review of this revision.Wed, Oct 30, 12:16 PM

Revision Contents
Changeset List

Path

Size

lib/

selectors/

chat-selectors.js

157 lines

Diff 45474

View Options

lib/selectors/chat-selectors.js

[lib] Inline createChatThreadItemNeeds ReviewPublicActions