[lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`
ClosedPublic
Actions

Authored by atul on Jun 1 2024, 10:00 PM.

Details

Reviewers

ashoat
ginsu
tomek
varun

Commits

rCOMMc2da721a842f: [lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`

Summary

This change is kind of useless. Thought I'd be able to do a lot more narrowing of types "along" the memberHasAdminPowers callstack, but this was pretty much it. Figured I may as well put it up.

Depends on D12265

Test Plan

NA, flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Jun 1 2024, 10:00 PM

Harbormaster completed remote builds in B29343: Diff 40823.Jun 1 2024, 10:17 PM

atul requested review of this revision.Jun 1 2024, 10:17 PM

ashoat accepted this revision.Jun 2 2024, 2:09 PM

This revision is now accepted and ready to land.Jun 2 2024, 2:09 PM

rebase and land

Harbormaster completed remote builds in B29390: Diff 40889.Jun 3 2024, 11:42 AM

Closed by commit rCOMMc2da721a842f: [lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock` (authored by atul). · Explain WhyJun 3 2024, 12:08 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMc2da721a842f: [lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`.

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

4 lines

Diff 40891

View Options

lib/shared/thread-utils.js

Show First 20 Lines • Show All 104 Lines • ▼ Show 20 Lines	type Props = {
+decompressSocketMessage: CompressedData => string,		+decompressSocketMessage: CompressedData => string,
// Redux dispatch functions		// Redux dispatch functions
+dispatch: Dispatch,		+dispatch: Dispatch,
+dispatchActionPromise: DispatchActionPromise,		+dispatchActionPromise: DispatchActionPromise,
// async functions that hit server APIs		// async functions that hit server APIs
+socketCrashLoopRecovery?: () => Promise<void>,		+socketCrashLoopRecovery?: () => Promise<void>,
// keyserver olm sessions specific props		// keyserver olm sessions specific props
+getInitialNotificationsEncryptedMessage?: (		+getInitialNotificationsEncryptedMessage?: (
		keyserverID: string,
options?: ?InitialNotifMessageOptions,		options?: ?InitialNotifMessageOptions,
) => Promise<string>,		) => Promise<string>,
};		};
type State = {		type State = {
+inflightRequests: ?InflightRequests,		+inflightRequests: ?InflightRequests,
};		};
class Socket extends React.PureComponent<Props, State> {		class Socket extends React.PureComponent<Props, State> {
state: State = {		state: State = {
▲ Show 20 Lines • Show All 700 Lines • Show Last 20 Lines

[lib] Narrow `threadInfo` argument of `threadFrozenDueToViewerBlock`ClosedPublicActions