[lib] Inline createChatThreadItem
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 10, 4:01 AM

	Unknown Object (File)
	Sat, Dec 7, 9:27 PM

	Unknown Object (File)
	Tue, Nov 26, 10:01 PM

	Unknown Object (File)
	Sat, Nov 23, 5:31 AM

	Unknown Object (File)
	Nov 23 2024, 2:35 AM

	Unknown Object (File)
	Nov 17 2024, 9:06 PM

	Unknown Object (File)
	Nov 14 2024, 4:52 PM

	Unknown Object (File)
	Nov 14 2024, 4:27 PM

View All 33 Files

Subscribers

None

Details

Reviewers

tomek

Commits

rCOMMfd009c90168e: [lib] Inline createChatThreadItem

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

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

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

tomek accepted this revision.Oct 31 2024, 5:35 AM

This revision is now accepted and ready to land.Oct 31 2024, 5:35 AM

Closed by commit rCOMMfd009c90168e: [lib] Inline createChatThreadItem (authored by ashoat). · Explain WhyOct 31 2024, 11:38 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMfd009c90168e: [lib] Inline createChatThreadItem.

Revision Contents
Changeset List

Path

Size

lib/

selectors/

chat-selectors.js

157 lines

Diff 45539

View Options

lib/selectors/chat-selectors.js

[lib] Inline createChatThreadItemClosedPublicActions