Paths

Table of Contentst

Differential D11916

[native] Consume `useThreadInChatList` in `native/ThreadSettings`
ClosedPublic
Actions

Authored by atul on Mon, May 6, 2:11 PM.

Tags

None

Referenced Files

	F1876983: D11916.id40183.diff
	Mon, May 27, 1:38 PM

	F1876365: D11916.id40182.diff
	Mon, May 27, 12:28 PM

	Unknown Object (File)
	Sun, May 12, 5:37 PM

	Unknown Object (File)
	Fri, May 10, 3:56 PM

	Unknown Object (File)
	Mon, May 6, 3:28 PM

	Unknown Object (File)
	Mon, May 6, 3:24 PM

	Unknown Object (File)
	Mon, May 6, 3:09 PM

Subscribers

None

Details

Reviewers

ashoat
ginsu
tomek
varun
will

Commits

rCOMMd90270ac68f8: [native] Consume `useThreadInChatList` in `native/ThreadSettings`

Summary

Similar to rest of stack, we want to replace usages of threadHasPermission with useThreadHasPermission which we're doing indirectly when we replace threadInChatList with useThreadInChatList.

Depends on D11915

Test Plan

flow + close reading for now

Logged value of isThreadInChatList and ensured it was same before/after change.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Mon, May 6, 2:11 PM

atul published this revision for review.Mon, May 6, 2:16 PM

Harbormaster completed remote builds in B28720: Diff 39874.Mon, May 6, 2:29 PM

flow + close reading for now

Sounds risky for a change that changes the flow.

native/chat/settings/thread-settings.react.js
1166 ↗	(On Diff #39874)	I find these names a bit confusing - they don't indicate that they are boolean flags. This might be expensive, but I think it would be better to put `is` somewhere inside them.

This revision is now accepted and ready to land.Tue, May 7, 1:53 AM

atul added a child revision: D11924: [lib] Introduce `useThreadsWithPermission` to filter `ThreadInfo`s.Tue, May 7, 9:21 AM

atul edited the test plan for this revision. (Show Details)Tue, May 14, 1:15 PM

address feedback (useThreadInChatList -> useIsThreadInChatList)

This revision was landed with ongoing or failed builds.Tue, May 14, 1:26 PM

Closed by commit rCOMMd90270ac68f8: [native] Consume `useThreadInChatList` in `native/ThreadSettings` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMd90270ac68f8: [native] Consume `useThreadInChatList` in `native/ThreadSettings`.

Harbormaster completed remote builds in B28912: Diff 40182.Tue, May 14, 1:35 PM

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

4 lines

native/

chat/

settings/

thread-settings.react.js

7 lines

web/

chat/

thread-list-provider.js

4 lines

Diff 40183

lib/shared/thread-utils.js

Loading...

native/chat/settings/thread-settings.react.js

Loading...

web/chat/thread-list-provider.js

Loading...